| • alexhollender_WMF | |
| May 21 2019, 11:46 PM |
| F29867667: IMG_2849.PNG | |
| Jul 25 2019, 3:01 AM |
| F29867676: IMG_2852.PNG | |
| Jul 25 2019, 3:01 AM |
| F29867701: reading-web-staging.wmflabs.org_wiki_Main_Page(iPhone 6_7_8).png | |
| Jul 25 2019, 3:01 AM |
| F29867695: reading-web-staging.wmflabs.org_wiki_Antonio_Vivarini(iPhone 6_7_8).png | |
| Jul 25 2019, 3:01 AM |
| F29867673: IMG_2851.PNG | |
| Jul 25 2019, 3:01 AM |
| F29867699: reading-web-staging.wmflabs.org_wiki_Musical_film(iPhone 6_7_8).png | |
| Jul 25 2019, 3:01 AM |
| F29843435: bellOutlined.svg | |
| Jul 22 2019, 8:03 PM |
| F29843437: search.svg | |
| Jul 22 2019, 8:03 PM |
Currently we're using an out-of-date, outlined version of the notification/bell icon (see OOUI icon sheet). Bringing it up-to-date will align it visually with the user icon we're adding to the header for AMC users.
| non-AMC | AMC |
 |  |
Icons:
Note: this update should apply to logged-out, logged-in, and logged-in AMC users (i.e. everybody)
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Goal | ovasileva | T198313 [GOAL] Advanced mobile contributions | ||
| Resolved | ovasileva | T210655 [EPIC] AMC Navigation | |||
| Resolved | ovasileva | T228916 Deploy AMC to all Wikipedias | |||
| Resolved | phuedx | T214540 [EPIC] AMC Navigation - user menu | |||
| Resolved | • Niedzielski | T224070 Update notifications & search icons (AMC & non-AMC) |
@Volker_E thanks for point that out, I've updated the image in the description to avoid confusion.
Change 521772 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Refresh the bell icon
@Volker_E I'm using the bell icon in core and this is what I get... which seems a little big:
@Jdlrobson as discussed in Slack we will actually be updating both the search and the bell icons as part of this task. I will update the task description and title accordingly.
Note: this update should apply to logged-out, logged-in, and logged-in AMC users (i.e. everybody).
@Volker_E I'm using the bell icon in core and this is what I get... which seems a little big:
Is this the desired result?
Is the SVG asset in core up to date?
From a conversation with @alexhollender, it's not the bell icon that's big, it's the other ones that are too small. They have not been updated to the latest, due to lack of design resources – didn't want to put 1 filled icon into header, where MinervaNeue went for a more subtle design approach before.
@Volker_E after talking to @Jdlrobson we think we should add the outline versions of those icons to OOUI and MWUI as variations of filled in one. then we can refer to them on the header to avoid future debt
@Nirzar Sounds good, can go out into next weeks' OOUI release which would be in production train from Tue 30 July on. If there's urgent time sensitivity we could add them in an extra release before next Tue, which would land train next week.
In any case, we need the icons production-ready prepared as SVGs to proceed.
@Volker_E can we add them in as an extra release before next Tue?
and what needs to be done to the SVG?
@Nirzar What I've said before – yes, we could. We'd need the outlined bell & updated search icon on the 20x20px canvas, and the outlined user icon, which is planned as well? So it's only the latter missing here right now from my understanding.
Change 524948 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] icons: Add 'bellOutline' and 'userAvatarOutline' and amend 'search'
Change 524948 merged by jenkins-bot:
[oojs/ui@master] icons: Add 'bellOutline' and 'userAvatarOutline' and amend 'search'
Change 524959 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Update OOUI to v0.33.4
Please design review on https://readingwebstaging.wmflabs.org/wiki/Main_Page prior to merge.
looks great. Will make a followup task for some slight tweaks but we're good to move forward. Here's what I'm seeing:
| logged out | logged in | AMC | |
| iOS |  |  |  |
| Android |  |  |  |
@Jdlrobson moving to signoff, however if you think there are reasons to QA this more extensively please move back to QA.
Change 521772 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Refresh the user, bell and search icon
