Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

Better notification support #199

Closed
eygraber opened this issue Mar 26, 2017 · 10 comments
Closed

Better notification support #199

eygraber opened this issue Mar 26, 2017 · 10 comments

Comments

@eygraber
Copy link
Contributor

Notifications look like they're being posted as multiple notifications, so if I have a lot of them it locks up the system notification drawer for a few seconds.

Group notifications or an inbox style would be better.

@kutsan
Copy link
Contributor

kutsan commented Mar 26, 2017

I guess you haven't updated yet to v1.4.0. As you can see at #111.

@eygraber
Copy link
Contributor Author

Are they being posted all at once, or in a batch? I have a few hundred unread notifications and when they get posted the notification drawer gets locked up.

@kutsan
Copy link
Contributor

kutsan commented Mar 26, 2017

They are now posting in a batch. Like this and this. I hope this is what you are asking for?

@eygraber
Copy link
Contributor Author

What I'm saying is that here each unread notification is posted as a Notification but if there are a lot of them, it will lock up the notification drawer. There should be some way to post them all at once, or stagger them so it doesn't lock up the UI.

@kutsan
Copy link
Contributor

kutsan commented Mar 26, 2017

cc @k0shk0sh

@k0shk0sh
Copy link
Owner

k0shk0sh commented Mar 26, 2017

I guess we should limit them to max 20 ? perhaps?

@kutsan
Copy link
Contributor

kutsan commented Mar 27, 2017

Sorry for misunderstanding, re-opening. We should limit them at some point.

@k0shk0sh
Copy link
Owner

@eygraber could you let me know what is your current OS version?
Because they are as @kutsan described, they are stacked as of Gmail style (Bundled together) as you see from here

this notification is serving as a grouped (mother) notification that host all of the other notifications that are being showed in the link you provided.

@eygraber
Copy link
Contributor Author

@k0shk0sh7.1.1 on a pixel

@k0shk0sh
Copy link
Owner

@eygraber I should have figured that out earlier as you are suffering the nightmode, which I just tested it out in 6.0.1 and it look amazing 😌😌.
alright then, I guess a limit to the notification should do.
thanks.

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

No branches or pull requests

3 participants