Page MenuHomePhabricator

Notification icon missing on cx skin
Closed, ResolvedPublicBUG REPORT

Description

we're missing the notification icon on cx/sx

cx skinvectorvector legacy
image.png (136×446 px, 5 KB)
image.png (186×596 px, 6 KB)
image.png (98×394 px, 5 KB)

Event Timeline

Pginer-WMF renamed this task from issue on cx skin to Notification icon missing on cx skin.Sep 27 2021, 11:35 AM
Pginer-WMF triaged this task as Medium priority.
Pginer-WMF edited projects, added ContentTranslation; removed SectionTranslation.

Change 724048 had a related patch set uploaded (by Santhosh; author: Santhosh):

[mediawiki/extensions/ContentTranslation@master] CX Skin: Fix missing icons in toolbar

https://gerrit.wikimedia.org/r/724048

Change 724048 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] CX Skin: Fix missing icons in toolbar

https://gerrit.wikimedia.org/r/724048

cx skin looks a bit different from both legacy vector and new vector, is this ok @Pginer-WMF ?

cxlegacy vectornew vector
Screenshot 2021-10-04 at 14.11.40.png (138×414 px, 10 KB)
Screenshot 2021-10-04 at 14.11.33.png (84×362 px, 10 KB)
Screenshot 2021-10-04 at 14.10.27.png (184×492 px, 10 KB)

cx skin looks a bit different from both legacy vector and new vector, is this ok @Pginer-WMF ?

cxlegacy vectornew vector
Screenshot 2021-10-04 at 14.11.40.png (138×414 px, 10 KB)
Screenshot 2021-10-04 at 14.11.33.png (84×362 px, 10 KB)
Screenshot 2021-10-04 at 14.10.27.png (184×492 px, 10 KB)

The ideal would be to converge towards the new vector over time. But that can be done in future tickets. For now, I think it makes sense to keep the task focused on restoring the missing functionality even if it is not aligned with the latest desktop improvements.