The chat number indicator in Discourse becomes illegible when there are more than 100 notifications. AhmedLoud suggested displaying “99+” when the count exceeds 100 and offered to create a pull request. However, awesomerobot and chapoi think a better solution would be to make the container grow to fit the content, similar to the style used elsewhere in Discourse. chapoi proposed setting a maximum count of “999+” and using adaptive styling. The issue has since been fixed in this pull request, as confirmed by chapoi.
I noticed that the chat number indicator looks broken when there is more than 100.
One solution could be to display 99+ when there is more that 100.
I will be happy to create a pull request if no one is working on it. The chat plugin is inside the discourse base code right ?
I think @chapoi has had some thoughts about this before… should it follow the style we use elsewhere where the container can grow to fit the content (and becomes more pill-shaped in the process?)
correct!
Yes, I think a combination would be best because even 99+ doesn’t fit.
So set something as the maximum and use the same styling as on the chat-index page on mobile so it adapts.
Perhaps it should be 999+ as maximum?
Coincidentally, I am just reworking the unread indicator! So this should be fixed soonish.
This has been fixed in
https://github.com/discourse/discourse/pull/21946
This topic was automatically closed after 4 days. New replies are no longer allowed.
