Notification badges blinking
Link issues together to show that they're related. Learn more.
Activity
- Author Contributor
@matthieu You might have an opinion on that one
By Alexandre on 2020-01-30T13:29:09 (imported from GitLab project)
- Maintainer
Probably related with this one startinblox/framework/sib-core#135?
The
sib-display
are empty while waiting for their data to be fetched. That's the blink we see here.I don't see a quick fix for now...
- Jean-Baptiste Pasquier changed milestone to %10
changed milestone to %10
- Author Contributor
@rachel, @jbpasquier Now that the chat experience is almost cool, the priority of this should be raised in my opinion. I find it more and more disturbing now that the rest is working :)
By Alexandre on 2020-02-28T06:33:04 (imported from GitLab project)
- Rachel changed milestone to %Circles part I
changed milestone to %Circles part I
- Alex changed title from Notification badges bliking to Notification badges blinking
changed title from Notification badges bliking to Notification badges blinking
- Alex mentioned in issue #446 (closed)
mentioned in issue #446 (closed)
- Rachel changed milestone to %Circles MVP
changed milestone to %Circles MVP
- Rachel changed milestone to %Chat (not in dedicated repo)
changed milestone to %Chat (not in dedicated repo)
- Jean-Baptiste Pasquier assigned to @matthieu
assigned to @matthieu
- Maintainer
Today we have a "sib way" to display the counter (one sib-display filtered on unread notifications and a counter).
Before the widget refactoring, if we want to get rid of this, we should probably go full js for the counter in order to stop it blinking. What do you think @balessan?
- Maintainer
Actually I may have found a better solution. But I don't have the right to merge the MR framework/sib-notifications!20 (merged)
- Maintainer
I like the sound of that.
@sylvain, @jbpasquier, @balessan, @clement Could one of you give the right access to @matthieu? I think you have as maintainers of the project.
- Maintainer
It's ok, thanks! @balessan do you want to test it before merging?
- Owner
I would prefer but I am short on time today. Can you specify the quickest test process in the MR @matthieu ?
- Maintainer
Sure!
- Please register or sign in to reply
- Maintainer
Should be fixed with the last MR of sib-notifications
- Maintainer
Very cool news. Never been a big one but it conveys a sketchy feeling about the app. It will be much better without it.
- Jean-Baptiste Pasquier closed
closed