|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
Created attachment 8716538 [details] Mozreview comparison mozscreenshots detected a change in the Hello toolbar icon on today's Nightly (2016-02-05) whereby the left, top and right of the toolbar button shadow/outline seems ever so slightly clipped. It seemed to only be detected with a dark theme. See the comparison at https://reviewboard.mozilla.org/r/33857/file/187-189/ Comparison between the Nightlies: http://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=f2f8fc172f4c&newProject=mozilla-central&newRev=1dbe350b57b1  https://github.com/mnoorenberghe/mozscreenshots
(In reply to Matthew N. [:MattN] from comment #0) > It seemed to only be detected with a dark theme. This is *particularly* noticeable if you have a "dark theme" that has a colorful background (rather than a black or gray background) -- sample screenshot: attachment 1246511. (In the dark lightweight theme screenshots at screenshots.mattn.ca URL in comment 0, this bug is harder to see, because the icon's cropped border just blends into the black/gray background around it.)
(er, not sure why my "attachment 1246511" text wasn't linkified. Anyway, that screenshot is https://bugzilla.mozilla.org/attachment.cgi?id=8716793 )
Sevaan, please can we get 16x16 / 32x32 versions of all the toolbar icons. We could do with them fairly quickly as the buttons we're currently broken in the inverted icons case (xref bug 1257672 comment 3). Although we could just get the inverted ones done, the non-inverted ones also are based around 18x18, and I think it would be good to just get them all resized correctly now. Our current versions are under in https://github.com/mozilla/loop/tree/master/add-on/chrome/skin
Created attachment 8732886 [details] Toolbar Assets Please let me know if these work, thanks!
This is being fixed by bug 1257672 which should land today, so I'll mark it as fixed now.
Not fixed for me. Build 2016-04-02
(In reply to Charles Milette from comment #8) > Not fixed for me. Build 2016-04-02 This is not in nightly builds yet. It has landed in our special loop repository, but hasn't moved across to mozilla-* yet.