Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need notifications #94

Closed
DuendeInexistente opened this issue Aug 7, 2022 · 11 comments · Fixed by #145
Closed

Need notifications #94

DuendeInexistente opened this issue Aug 7, 2022 · 11 comments · Fixed by #145

Comments

@DuendeInexistente
Copy link

DuendeInexistente commented Aug 7, 2022

It's the one glaringly missing feature, notifications. Also an Unread Messages banner on the server list like Discord has.

@ouwou
Copy link
Member

ouwou commented Aug 7, 2022

whats the unread messages banner? the NEW thing?

@DuendeInexistente
Copy link
Author

Yeah the NEW thing.

@dragontamer8740
Copy link
Contributor

dragontamer8740 commented Aug 13, 2022

I kind of would prefer not to have notifications... or if we have them, the ability to turn them off would be appreciated. That's one of the things that drives me crazy about the official client.

Nothing wrong with having an option for them though.

@ouwou
Copy link
Member

ouwou commented Aug 13, 2022

i think itll be both compile time and runtime optional if i get to it since im probably going to have to drag in another dependency anyways

@KaKi87
Copy link

KaKi87 commented Feb 3, 2023

Hello,

Any news on this ?

This is actually critical, because I've discovered the worst way, that when Abaddon is connected, I won't receive mobile notifications either, since my computer is (rightfully) considered to be my currently active device.

Thanks

@ouwou
Copy link
Member

ouwou commented Feb 7, 2023

Hello,

Any news on this ?

This is actually critical, because I've discovered the worst way, that when Abaddon is connected, I won't receive mobile notifications either, since my computer is (rightfully) considered to be my currently active device.

Thanks

you can probably mitigate it by changing the timeout to 1 minute in the notifications settings. and when we say notifications do we want just sounds or fully fledged desktop notifications?

@KaKi87
Copy link

KaKi87 commented Feb 7, 2023

changing the timeout to 1 minute in the notifications settings

Where do I find that option ?

fully fledged desktop notifications?

That.

@ouwou
Copy link
Member

ouwou commented Feb 7, 2023

Where do I find that option ?

official discord app > notifications > push notification inactive timeout
dont know if it will work well but its something for the time being i guess

@ouwou ouwou mentioned this issue Mar 12, 2023
8 tasks
@KaKi87
Copy link

KaKi87 commented Apr 18, 2023

That's not a desktop notification 😅

Screenshot from 2023-04-18 05-21-06

Also, it would be nice to focus the app and open the channel when clicking it.

Thanks

@ouwou
Copy link
Member

ouwou commented Apr 18, 2023

That's not a desktop notification 😅

Screenshot from 2023-04-18 05-21-06

Also, it would be nice to focus the app and open the channel when clicking it.

Thanks

notifications are implemented with Gio, which in practice will use the freedesktop.org spec (which by the specification should specifically not look like your image lol), so ultimately its up to your desktop environment to determine what that looks like. maybe if you tell me which DE/notification server youre using i can try to help but i dont think theres a lot i can do code-wise.

also, it will bring you to the channel when clicking on it assuming your notification server supports actions

@KaKi87
Copy link

KaKi87 commented Apr 18, 2023

tell me which DE/notification server youre using

My desktop environment is Unity 7.6, on the Ubuntu Unity 22.04 distro.

I don't know about notification server.

I'll try on another computer using Cinnamon on Linux Mint 21, see if it's any better.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants