Page MenuHomePhabricator
Create Task
Maniphest T140900

Change bubble speech icon to tray icon
Closed, ResolvedPublic
Actions

Assigned To
Mooeypoo
Authored By
Trizek-WMF
Jul 20 2016, 2:52 PM
Tags
Referenced Files
F4346448: Screen Shot 2016-08-08 at 7.33.29 PM.png
Aug 9 2016, 2:44 AM
F4346454: Screen Shot 2016-08-08 at 7.29.33 PM.png
Aug 9 2016, 2:44 AM
F4346447: Screen Shot 2016-08-08 at 7.33.48 PM.png
Aug 9 2016, 2:44 AM
F4326825: echo icon overlap.png
Aug 1 2016, 4:41 PM
F4319460: Screen Shot 2016-07-29 at 5.42.46 PM.png
Jul 30 2016, 1:15 AM
F4319467: Screen Shot 2016-07-29 at 5.28.34 PM.png
Jul 30 2016, 1:15 AM
F4319462: Screen Shot 2016-07-29 at 5.42.57 PM.png
Jul 30 2016, 1:15 AM
F4319469: Screen Shot 2016-07-29 at 5.24.26 PM.png
Jul 30 2016, 1:15 AM
View All 10 Files
Subscribers
Aklapper
Catrope
Etonkovidova
gerritbot
• jmatazzoni
LikeLifer
• Mattflaschen-WMF
View All 12 Subscribers
Tokens
"Like" token, awarded by RandomDSdevel."Love" token, awarded by matej_suchanek.

Description

Per that comment T135114#2476359 and the following, it is now time to change the bubble speech icon by the tray icon to remove the idea of "messages" when you click on the "notices" badge.

I've set priority as high, because CLs receive feedback about that inconstancy.

The tray icon is still on discussion for polishing: T135114: Review "tray" icon - that discussion is stalled at the moment because of vacations. When the polishing is done, the icon will be changed again.

As a reminder, the new icon:

tray.svg2 KBDownload

Details

SubjectRepoBranchLines +/-
Redo the notification badgesmediawiki/extensions/Echomaster+186 -211
icons: Provide a 'tray' icon in alerts packoojs/uimaster+8 -2
Customize query in gerrit

Related Objects
Search...

Event Timeline

Trizek-WMF created this task.Jul 20 2016, 2:52 PM
Trizek-WMF moved this task from Untriaged to Ready for Pickup on the Collab-Team-Q1-July-Sep-2016 board.
gerritbot added a comment.Jul 20 2016, 9:56 PM

Change 298030 had a related patch set uploaded (by Jforrester):
icons: Provide a 'tray' icon in alerts pack

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

gerritbot added a project: Patch-For-Review.Jul 20 2016, 9:56 PM
gerritbot added a comment.Jul 20 2016, 10:05 PM

Change 298030 merged by jenkins-bot:
icons: Provide a 'tray' icon in alerts pack

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

Jdforrester-WMF removed a project: Patch-For-Review.Jul 20 2016, 10:34 PM
matej_suchanek added a project: User-notice.Jul 21 2016, 7:31 AM
Trizek-WMF moved this task from To Triage to Not ready to announce on the User-notice board.Jul 21 2016, 7:50 AM
Trizek-WMF moved this task from Not ready to announce to In current Tech/News draft on the User-notice board.
Trizek-WMF updated the task description. (Show Details)Jul 21 2016, 7:54 AM
gerritbot added a comment.Jul 27 2016, 1:03 AM

Change 299910 had a related patch set uploaded (by Mooeypoo):
Redo the notification badges

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

Johan moved this task from In current Tech/News draft to Recently announced in Tech/News on the User-notice board.Jul 27 2016, 5:33 PM
LikeLifer subscribed.Jul 28 2016, 1:40 PM
Mooeypoo claimed this task.Jul 28 2016, 10:51 PM
Mooeypoo moved this task from Ready for Pickup to Needs Review on the Collab-Team-Q1-July-Sep-2016 board.
Trizek-WMF mentioned this in T140645: Review if "notices" are not translated by "messages" for Notifications fly-outs on TranslateWiki.Jul 29 2016, 9:00 AM
Trizek-WMF added a subtask: T140645: Review if "notices" are not translated by "messages" for Notifications fly-outs on TranslateWiki.
gerritbot added a comment.Jul 29 2016, 10:53 PM

Change 299910 merged by jenkins-bot:
Redo the notification badges

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

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-08-02_(1.28.0-wmf.13)).Jul 29 2016, 11:01 PM
Mooeypoo moved this task from Needs Review to QA Review on the Collab-Team-Q1-July-Sep-2016 board.Jul 29 2016, 11:46 PM
Etonkovidova added a comment.Jul 30 2016, 1:15 AM

Checked in betalabs 1.28.0-alpha (4c7cd24).
@Pginer-WMF - need your feedback on the following:

Some screenshots

Screen Shot 2016-07-29 at 5.42.46 PM.png (130×607 px, 26 KB)

Closer look:
Screen Shot 2016-07-29 at 5.42.57 PM.png (84×393 px, 13 KB)

Double digits for number of notifications:
Screen Shot 2016-07-29 at 5.28.34 PM.png (55×380 px, 13 KB)

Only Notices are present:

Screen Shot 2016-07-29 at 5.46.56 PM.png (228×560 px, 24 KB)

Only read Notices and Alerts:

Screen Shot 2016-07-29 at 5.24.26 PM.png (95×544 px, 21 KB)

Etonkovidova moved this task from QA Review to Product Review on the Collab-Team-Q1-July-Sep-2016 board.Jul 30 2016, 1:16 AM
Etonkovidova mentioned this in T115845: Notification panel: Clearer use of notification badges.Jul 30 2016, 1:22 AM
matej_suchanek awarded a token.Jul 30 2016, 7:13 AM
• Quiddity subscribed.Aug 1 2016, 4:41 PM

I see an alignment problem for the "99+ notices" scenario:

echo icon overlap.png (258×321 px, 8 KB)

• jmatazzoni subscribed.Aug 2 2016, 12:56 AM

@Etonkovidova, I heard some talk today about Nick's comment re. 99+ above. Before I close this, was that turned into a separate ticket?

RandomDSdevel awarded a token.Aug 5 2016, 10:53 PM
• jmatazzoni moved this task from Product Review to QA Review on the Collab-Team-Q1-July-Sep-2016 board.Aug 8 2016, 6:53 PM
Etonkovidova added a comment.Aug 9 2016, 2:44 AM

@jmatazzoni - I filed it as a separate issue T142454: Notices tray icon with 99+ needs more space since it's a specific bug.
'99+' case (since it occupies three spaces) looks a little bit tight:

Some examples (the browser display scale is 110%):

Screen Shot 2016-08-08 at 7.33.48 PM.png (100×650 px, 21 KB)

Screen Shot 2016-08-08 at 7.29.33 PM.png (120×829 px, 28 KB)

Monobook skin:

Screen Shot 2016-08-08 at 7.33.29 PM.png (56×698 px, 28 KB)

Etonkovidova moved this task from QA Review to Product Review on the Collab-Team-Q1-July-Sep-2016 board.Aug 9 2016, 2:45 AM
Trizek-WMF added a comment.Aug 9 2016, 1:04 PM

There is comments about the tray icon. Most of time people don't understand the meaning of that icon.

  • on en.wp ("bikini top")
  • on fr.wp ("rear of my car")
  • on de.wp ("subway face")
• jmatazzoni closed this task as Resolved.Aug 9 2016, 6:15 PM
• jmatazzoni mentioned this in T142511: Evaluate design of notifications inbox (“tray”) icon.

OK, I'm closing this but have opened a separate ticket to evaluate icon design, T142511.

Liuxinyu970226 unsubscribed.Aug 9 2016, 9:26 PM
Johan moved this task from Recently announced in Tech/News to Already announced/Archive on the User-notice board.Aug 10 2016, 2:27 AM
Trizek-WMF closed subtask T140645: Review if "notices" are not translated by "messages" for Notifications fly-outs on TranslateWiki as Resolved.Aug 10 2016, 12:17 PM
Ladsgroup edited projects, added User-notice-archive; removed User-notice.Aug 13 2022, 1:53 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits