Needinfo shorlander to provide the modified icons.

Change icons to 16 x 16

Comment on attachment 8865653 [details] Bug 1362083 - Change toolbarbutton icon size from 18x18 to 16x16. https://reviewboard.mozilla.org/r/137270/#review140616 This seems good to me but there's an issue with the downloads indicator that I just mentioned to you (it gets larger on click), not sure if that's trivial to fix or not, so cancelling review for now.

Comment on attachment 8865653 [details] Bug 1362083 - Change toolbarbutton icon size from 18x18 to 16x16. https://reviewboard.mozilla.org/r/137270/#review140626 Looks good now, thanks.

Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d3c404886f0f Change toolbarbutton icon size from 18x18 to 16x16. r=johannh

Mozscreenshots found that this patch (at least I think it was this patch from looking at the pushlog) slightly stretched the back-button icon. https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=4b8939ed128172fcd3433ee25f336a8c1ba77ab0&newProject=mozilla-central&newRev=2b6f6881a24a726364054aa69031927e2f3fc5fa I guess it doesn't make a big difference visually and compact theme seems to be fine, I just wanted to note this. I'm not really sure how to follow up to this.

(In reply to Johann Hofmann [:johannh] from comment #10) > Mozscreenshots found that this patch (at least I think it was this patch > from looking at the pushlog) slightly stretched the back-button icon. > > https://screenshots.mattn.ca/compare/?oldProject=mozilla- > central&oldRev=4b8939ed128172fcd3433ee25f336a8c1ba77ab0&newProject=mozilla- > central&newRev=2b6f6881a24a726364054aa69031927e2f3fc5fa > > I guess it doesn't make a big difference visually and compact theme seems to > be fine, I just wanted to note this. I'm not really sure how to follow up to > this. Icons generally shouldn't be stretched. Can you please file a new bug?

Done :)