| Etonkovidova | |
| Mar 24 2017, 7:30 PM |
| F27009870: image.png | |
| Nov 1 2018, 3:23 PM |
| F18601400: Screen Shot 2018-05-29 at 12.55.03 PM.png | |
| May 29 2018, 7:58 PM |
| F6934434: Screen Shot 2017-03-24 at 8.49.27 AM.png | |
| Mar 24 2017, 7:30 PM |
It's really minor - the small 'tails' of the notification icons have different "direction". It's quite noticeable when they are displayed in the Notification panel.
Repo: https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/Flow
Directory: modules/notification/icons
Correct icons: M229
Source files: ???
Size: ???
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Notification: Add and make use of RTL icons | mediawiki/extensions/Flow | master | +73 -22 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Esanders | T183749 Align notification icons with the icon styleguide | |||
| Resolved | Volker_E | T161330 Flow: Notification icons point to different directions |
there's nothing really wrong with this. But it is curious, so we wanted to make sure it is planned (or fix if not).
There is an initiative in progress to update the general icon set (M229) to better align to the design guidelines. Once the process is complete it makes sense to adjust the notification-related icons to match the style, since especially the speech bubbles are derived from those.
@Pginer-WMF, given that the new speech bubble icon hasn't seen any questions or revisions do you think we are good to update the notifications icons. This might be a good task for Google-Code-in-2017 too.
The speech bubble icon seems stable, and I think it would be safe to use it as a base. Having said that, it is also possible that once the new speech bubble icon becomes the default and reaches all products, we identify further adjustments to be done. I'd not consider this a blocker, just something to be aware of when creating the new icons.
Could someone clarify what is wanted here, so this could maybe become a Google-Code-in-2017 task? Is this just about mirroring one icon?
The current issue reflected one of the several adjustments needed to align the icons shown in the notifications with the general set of icons. I created a more general task to capture it better (T183749). The work in that front is already in the review stage, so I don't think it is apt for Google-Code-in-2017.
Note also that only some of the speech bubble icons have RTL variants, so in RTL it will still be inconsistent after the parent bug is fixed (T183749).
Checked along with T183749: Align notification icons with the icon styleguide - the look has been improved due to new design, but @Esanders is right - several icons still need their RTL variants:
Still an issue
@Volker_E I'm not sure who to tag for creating RTL variants of some of the icons for Echo (i.e. /modules/icons/mention-success-constructive.svg), if not you, could you please point me to who I should tag/assign for creating these?
Change 489783 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[mediawiki/extensions/Flow@master] Notification: Add and make use of RTL icons
@kostajh Stumbled upon this again, RTL icons created and added. Please have a look…
For future reference, you can tag me, UI-Standardization or Design for those requests…
For future reference, you can tag me, UI-Standardization or Design for those requests…
Got it, thanks @Volker_E !
Change 489783 merged by jenkins-bot:
[mediawiki/extensions/Flow@master] Notification: Add and make use of RTL icons