it seems that OOUI icons have this problem too:

Should we add some IE8 specific rule to hide icons in this case to avoid the ugliness?

From my tests with BrowserStack this isn't caused by background-size as the hamburger icon and the search icon (both with background-size properties) are rendered correctly. Page requested: https://doc.wikimedia.org/oojs-ui/master/demos/demos.php?page=widgets&theme=wikimediaui&direction=ltr
It seems to be more a problem around the way the menu is constructed.
But the menu doesn't work anyways correctly in IE 8 (you can't close it anymore without clicking one of the items), so I wonder if it's worth tracking down…?

IE8 uses PNGs.
The main menu uses 24X24px https://en.m.wikipedia.org/w/load.php?modules=skins.minerva.icons.images&image=mainmenu&format=rasterized&lang=en&skin=minerva

but the main menu icons use 48X48
https://en.m.wikipedia.org/w/load.php?modules=mobile.mainMenu.icons&image=profile&format=rasterized&lang=en&skin=minerva&version=0aye6lb

Can we resize these safely....?

@Nirzar can we standardise on 24x24?

We have come to an agreement for 24px canvas with design team.

@Jdlrobson do you need the updated SVGs?

@Nirzar it would certainly make it easier if you can! Not sure if you can find and replace these SVGs and devs may not have SVG resize tools.

@Nirzar can you upload an updated version of the Anonymous icon too? That's the only one missing.

I also noticed that some of the new icons are completely different from the existing ones, while others have different colors. This change isn't mentioned in the task description. Can you confirm that it's the desired behavior?

Change 364509 had a related patch set uploaded (by Bmansurov; owner: Bmansurov):
[mediawiki/extensions/MobileFrontend@master] WIP: Update main menu icons

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

@Jdlrobson the proposed solution doesn't seem to fix the issue in IE8. Anything else that needs to be done?

@bmansurov I was kinda hoping to get rid of the logic to show different icons for logged in and logged out users. just show single icon for all but that would be adding to the scope?

Yes, that's a different issue that needs to be tackled separately if possible.

@bmansurov okay i will give another icon then. thanks

I also noticed that some of the new icons are completely different from the existing ones, while others have different colors. This change isn't mentioned in the task description. Can you confirm that it's the desired behavior?

I will mention it. WMF Design is under process to renew the icons to get rid of the teardrops and I would like to get that change in instead of updating the icons again in a month.

the color change was intended to be done last time around and we didn't do it because there is a technical debate on if we can change svg colors in code. while the debate continues, we shipped black icons and I would like to fix that.

I will update the task desc.

@bmansurov

here's the missing SVG for non logged in users

after testing it on RWS i caught a subpixel issue in the home icon. i fixed it -

Note: both SVGs have height width props in there

Talked with @bmansurov post-merge and we estimated this as a 1-2 (went for the larger more conservative estimate). We should check this in IE8 to sign off.

Change 364509 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Update main menu icons

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

@Jdlrobson does it need QA?

Tested on the beta cluster. The icons are no longer stretched.

Noticed a problem with the Wikipedia beta logo though...

yay!

Noticed a problem with the Wikipedia beta logo though...

That's captured in T166684 but thanks for checking!

@Nirzar did you want some feedback on Opera Mini/UTC or did we decide to do that later as part of a bigger analysis of those browsers? If not feel free to sign off.