993 Members
0.8.1 out now! https://github.com/Nheko-Reborn/nheko/releases/tag/v0.8.1 Discussion about nheko, the desktop client for Matrix :  https://github.com/Nheko-Reborn/nheko Latest Nightlies: https://matrix-static.neko.dev/room/!TshDrgpBNBDmfDeEGN:neko.dev/319 Servers

Load older messages

10 Mar 2021
@vurpo:hacklab.fitildeFedora 3310:20:00
@deepbluev7:neko.devNico|devRight, I'll try it out there10:20:14
@vurpo:hacklab.fitildeThe "quit after doing something in the background" functionality is a bit complex to describe10:28:38

My current idea is to set a flag at startup if it's started up in "push message" mode, and when that flag is set, it sets a 1 minute (maybe?) timeout and quits when that timeout runs out if the flag is set, and also quits the app at the end of ChatPage::sendNotifications if the flag is set

Then, if the window is brought up by the user (MainWindow::show), then the flag gets unset so the app doesn't quit automatically anymore

@deepbluev7:neko.devNico|devSounds reasonable10:34:44
@vurpo:hacklab.fitildethat timeout to quit is because there are lots of situations where a push message arrives but there are no notifications to process10:34:50
@vurpo:hacklab.fitilde(for example network error, or there simply aren't any messages to notify about)10:36:10
@deepbluev7:neko.devNico|devOr if the push was delayed and processed with a previous push :310:36:44
@vurpo:hacklab.fitildeAlso, push messages are also used to tell the app when there are no longer any messages to notify about, so nheko could technically delete desktop notifications that it has already sent, although that would require starting to keep track of notifications in the app database across processes, and that would be complicated10:37:22
@vurpo:hacklab.fitildeso I'm not going to try implementing that10:37:28
@deepbluev7:neko.devNico|devYeah, we should do that at some point, but you don't have to implement, since it is generalized across all notifications10:38:15
@vurpo:hacklab.fitildeAlso I noticed the app doesn't get brought up for me when clicking a notification, so I'm going to try seeing why that is10:38:59
@vurpo:hacklab.fitildethat is, the window isn't made visible10:39:25
@deepbluev7:neko.devNico|devHm, weird, I never run hidden window mode, so maybe that never worked >.<10:39:47
@vurpo:hacklab.fitildeAlso the notification has a blank button attached to it, so maybe the dbus notification api is just being called incorrectly10:40:15
@vurpo:hacklab.fitildeI'll look at that10:40:18
@deepbluev7:neko.devNico|devI feel like the blank button is the notifications reinterpreting the reply action wrongly maybe?10:41:01
        // The list of actions has always the action name and then a localized version of that
        // action. Currently we just use an empty string for that.
        // TODO(Nico): Look into what to actually put there
@vurpo:hacklab.fitildeI feel like this is why there's a blank button10:42:13
@deepbluev7:neko.devNico|devOh, maybe10:42:28
@deepbluev7:neko.devNico|devBut I'm pretty sure that an empty string should be allowed ._.10:42:43
@vurpo:hacklab.fitildeIt sure seems to be allowed10:42:59
@vurpo:hacklab.fitildebut it's literally a blank string, so that's what gets put on the button10:43:12
@deepbluev7:neko.devNico|devWell, you can try putting something else there, if that changes things, we have to translate it10:43:31
@deepbluev7:neko.devNico|devI guess all my systems don't show a button for the actions or provide their own translations :D10:48:06
@vurpo:hacklab.fitilde It's also not an inline reply field, even though I think Gnome supports inline reply in notifications 10:48:18
In reply to @vurpo:hacklab.fi
It's also not an inline reply field, even though I think Gnome supports inline reply in notifications
I'm pretty sure it doesn't
@deepbluev7:neko.devNico|devThe vetoed the proposal, iirc10:48:43
@vurpo:hacklab.fitildeI've never had an app with notifications with inline reply, but all the references I see online suggest that it does10:48:49
@vurpo:hacklab.fitilde activateWindow(); did not make the window appear after it was hidden, but window()->show(); did (in the notificationClicked handler in ChatPage) 15:13:17

There are no newer messages yet.

Back to Room ListRoom Version: 5