Page MenuHomePhabricator

Use single Echo badge for notifications with different colours to indicate status/type
Closed, DuplicatePublic

Description

Two badges presents problems with stuff fitting and being able to easily toggle between them, especially on mobile.

Also some people can't even figure out what the non-bell even is, whereas the bell is pretty common across devices/sites...

Event Timeline

kostajh subscribed.

@Isarra do you plan to work on this? I've moved it to External on the Growth-Team under that assumption.

@Isarra do you plan to work on this? I've moved it to External on the Growth-Team under that assumption.

Let's put that down as a definite 'maybe'. Whatever the case, these two skins in particular could really use this, and it might be nice if we could backport whatever Minerva's doing for the lot of them, as it likewise only has a single icon...

@Jdlrobson, can you provide any insight into that? Do you reckon it's a good example that could translate to more usage?

We would like to move all mobilefrontends echo code into the Echo extension. It's a continued annoyance that when working on Echo we have to work in Echo, MF and Minerva so I think this needs to happen first. It's code then possibly needs to be rewritten to OOUI or no framework. At that point it's probably suitable for research. Personally I struggle to justify thr fact we have two implementations of the Echo feature.

Consider following my approach in https://gerrit.wikimedia.org/r/530964 for Timeless- you can wire up just one notification window using desktop Echo

We would like to move all mobilefrontends echo code into the Echo extension. It's a continued annoyance that when working on Echo we have to work in Echo, MF and Minerva so I think this needs to happen first.

Yeah, agreed. That's on Growth-Team's agenda for an upcoming sprint T221007: Extract Echo handling from SkinMinerva class.

T142981 is essentially a duplicate of this ticket. Not sure which way to merge.