@alexhollender Ok, if that's the case I'll just remove the opacity, which will look like this: https://jmp.sh/hpK2Jte
I made a mistake earlier, both echo icons and the user menu icons are actually just black #000, so I'll just leave them as be until that other color task

@Jdlrobson Yeah I could put up a follow up patch for Echo. I was thinking it might be easier to disable the opacity entirely for user links, and just set the color explicitly, what do you think?

In T285259#7173683, @Jdlrobson wrote:

I suggest we use this ticket to focus on getting Echo's icons aligned with Vectors (in whatever form makes sense). I would suggest it makes sense for Echo to register menu items with icon class and for the classes to be added on the Vector side just as we have done with the user menu.

@Jdlrobson I think this task isnt done yet, as this patch hasnt been merged yet
https://gerrit.wikimedia.org/r/c/mediawiki/skins/Metrolook/+/700934

@alexhollender I made this patchdemo that adds an additional 5 second delay. Let me know if this works for review!
https://patchdemo.wmflabs.org/wikis/3a8e8101b1/wiki/Main_Page?useskinversion=2

@Edtadros Ah that's my bad, I fixed the login/logout button but I forgot to remove the Login, Talk, and Contributions link for logged out. I'll put in another patch for that

@Jdlrobson Yep can do!

I also see an instance of the mixin in ULS, should that be added to this task?
https://github.com/wikimedia/mediawiki-extensions-UniversalLanguageSelector/blob/master/resources/css/ext.uls-vector.less#L8

@Jdlrobson Should we also hard deprecate getAfterPortlet? The release notes deprecate both getAfterPortlet and renderAfterPortlet together in 1.35:

An update on remaining work:

1. MERGED Storybook https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/696651
2. MERGED Follow up to comments https://gerrit.wikimedia.org/r/686781
3. Remove UserMenu template https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/698587

Hi @Edtadros, I just took a second look at the AC3 on prod, and it seemed to work for me. Does this task still need work? https://jmp.sh/PdhIcyR

I was looking at this again and I realized I made a mistake earlier, the landmark is being announced correctly with Chrome/JAWS. This issue is limited to Chrome/VO afterall

I did some additional testing, and it seems this is only happening with Chrome/VO and Chrome/Jaws. I tested Safari/VO, Firefox/Narrator, and Chrome/Narrator. In Chrome/Jaws it's not announced as a banner at all when navigating through landmarks and when listing all landmarks.

@alexhollender The mocks seem to have a minimum width or width, do you have a particular value in mind? I've been using width: 12em or 192px, which looks like this: https://jmp.sh/Bz4MeqL

@Jdlrobson Yes, I think we could definitely unset the logout link in onSkinTemplateNavigation in Hooks.php, and handle it manually. I was also thinking an even simpler solution could be to just reorder $content_navigation['user-menu']['logout'] in that hook instead of unsetting it.

@alexhollender I went ahead and made a spike ticket to get started on the sign off criteria, please feel free to edit it! https://phabricator.wikimedia.org/T282480