The icons of the accounts in the menu are blurry on a Mac with Retina display because the icons are not HiDPI. STR: In the menu go to "Go -> Folder" or to "File -> Get new messages for"

I've also changed some LoDPI icons to match the folder pane and fixed a bug in folderPane.css.

Comment on attachment 782251 [details] [diff] [review] Add HiDPI to folderMenu.css Redirecting to mconley, because I (still) don't have a Retina Mac… ;)

Hm, I don't know why, but thats not working for me, the icons are looking the same after this patch on my rMBP.

(In reply to Nomis101 from comment #4) > Hm, I don't know why, but thats not working for me, the icons are looking > the same after this patch on my rMBP. Is this again in main menu? Can you check the context- and the AppButton menu? Also in Folder Location toolbar menulist item you can check the icons. If it's only in main menu it could be a toolkit bug (I don't know if Firefox is using HiDPI icons in main menu).

(In reply to Richard Marti [:Paenglab] from comment #5) > (In reply to Nomis101 from comment #4) > > Hm, I don't know why, but thats not working for me, the icons are looking > > the same after this patch on my rMBP. > > Is this again in main menu? Can you check the context- and the AppButton > menu? Also in Folder Location toolbar menulist item you can check the icons. > > If it's only in main menu it could be a toolkit bug (I don't know if Firefox > is using HiDPI icons in main menu). Yes its again in the main menu. I've checked the context- and the AppButton menu and it looks not blurry and totaly fine there.

(In reply to Richard Marti [:Paenglab] from comment #5) > (I don't know if Firefox > is using HiDPI icons in main menu). Yes, they do and this icons are also affected. So I think you are right and this is a toolkit bug. I opened Bug 899334 for that.

Hey Richard, I applied the patch, but in the OS X menus, I'm still seeing pixel-ated sprites. I can tell via DOMi that the right CSS is being applied... and the sprites look right. What gives? Did we forget to do something that makes the hidden window pick up the changes to the menubar? Are you able to see the pixelation still? -Mike

Comment on attachment 782251 [details] [diff] [review] Add HiDPI to folderMenu.css Dropping review requests until we can get my above concern sorted.

Comment on attachment 782251 [details] [diff] [review] Add HiDPI to folderMenu.css Yes, like in comment 4 to 7 written, it looks like a toolkit bug (bug 899334). The exactly same rules applied on context, AppMenu and button-getMsgPopup are looking correct. I have no Retina Mac to test this and find a solution for the OS X menus. Re-adding the reviews as this can't fixed by this bug and the icons are looking correct in all other menus.

Clearing the needinfo with the answer in comment 10.

Comment on attachment 782251 [details] [diff] [review] Add HiDPI to folderMenu.css Review of attachment 782251 [details] [diff] [review]: ----------------------------------------------------------------- Just a suggestion below - no biggie if there's something that restricts you from doing it successfully. If it can't be done, land away. Thanks! ::: mail/themes/osx/mail/folderMenus.css @@ +185,5 @@ > + /* ..... servers ..... */ > + > + /* IMAP servers */ > + .folderMenuItem[IsServer="true"][ServerType="imap"][IsSecure="true"] { > + list-style-image: url("chrome://messenger/skin/icons/server@2x.png"); Is it possible to not repeat the list-style-image property each time, and have it applied to any .folderMenuItem where IsServer="true"? And the same for folder-pane@2x.png - can we do that for .folderMenuItems where SpecialFolder != ""? No biggie if there's a good reason not to, it just looks repetitive here.

Removed the repeated list-style-images in folderMenus.css. I checked the new file locally and every icon is shown correctly.