Dão Gottwald [:dao] Reporter
	Description • 8 years ago

Bug 1347543 switches us from Toolbar.png to SVGs, so later we should be able to drop in new icons without much manual CSS work.

	Dão Gottwald [:dao] Reporter
	Comment 3 • 7 years ago

Nihanth, could you please prepare a patch for this? Would be nice to land this straight after 56 moves to beta.

	Dão Gottwald [:dao] Reporter
	Comment 7 • 7 years ago
mozreview-review

Comment on attachment 8892119 [details]
Bug 1355455 - Replace shared icons with Photon versions.

https://reviewboard.mozilla.org/r/163108/#review168420

	Mozilla Autoland
	Comment 8 • 7 years ago

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 4ddd90d91cdc -d f91c159b3170: rebasing 411079:4ddd90d91cdc "Bug 1355455 - Replace shared icons with Photon versions. r=dao" (tip)
merging browser/themes/shared/customizableui/customizeMode.inc.css
merging browser/themes/shared/icons/device-desktop.svg
merging browser/themes/shared/icons/device-mobile.svg
merging browser/themes/shared/icons/email-link.svg
merging browser/themes/shared/jar.inc.mn
merging browser/themes/shared/tabs.inc.css
warning: conflicts while merging browser/themes/shared/icons/device-desktop.svg! (edit, then use 'hg resolve --mark')
warning: conflicts while merging browser/themes/shared/icons/device-mobile.svg! (edit, then use 'hg resolve --mark')
warning: conflicts while merging browser/themes/shared/icons/email-link.svg! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

	Pulsebot
	Comment 10 • 7 years ago

Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5c84321d226e
Replace shared icons with Photon versions. r=dao

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 11 • 7 years ago

Backed out for failures like https://treeherder.mozilla.org/logviewer.html#?job_id=120535233&repo=autoland

https://hg.mozilla.org/integration/autoland/rev/c7fda3c9ff866a9c830f8a91068af3e27a64238a

	(inactive) Nihanth Subramanya [:nhnt11] Assignee
	Comment 13 • 7 years ago

A whitelisted image was renamed; I updated the entry in the test file and re-landed.

	Pulsebot
	Comment 14 • 7 years ago

Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/c9c1ce4407b9
Replace shared icons with Photon versions. r=dao

	Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)
	Comment 15 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/c9c1ce4407b9

	Johann Hofmann [:johannh]
	Comment 16 • 7 years ago

Screenshots:

https://screenshots.mattn.ca/compare/?oldProject=mozilla-central&oldRev=d942ef54fdf726840a698d2ddcaf989d2c00edbc&newProject=mozilla-central&newRev=681a2489f6a92a09f87336c4ebfcd55030ffcda1

(We're still working on showing Windows comparisons after the taskcluster switch, you can check out the Windows screenshots here: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=681a2489f6a92a09f87336c4ebfcd55030ffcda1&filter-tier=1&filter-tier=2&filter-tier=3&exclusion_profile=false&filter-searchStr=ss&selectedJob=120695185)

	ovidiu boca[:Ovidiu]
	Comment 17 • 7 years ago

Hi Nihanth,

I have a question about this issue, I see on the screenshots that the "Pocket" should be near the menu icon, but on Nightly 57.0a1(2017-08-07) and on mockup page: http://design.firefox.com/people/shorlander/photon/Mockups/macOS.html the "Pocket" icon is in the address bar. Can you please clarify where this should be? Thanks

	Dão Gottwald [:dao] Reporter
	Comment 18 • 7 years ago

(In reply to ovidiu boca[:Ovidiu] from comment #17)
> Hi Nihanth,
> 
> I have a question about this issue, I see on the screenshots that the
> "Pocket" should be near the menu icon, but on Nightly 57.0a1(2017-08-07) and
> on mockup page:
> http://design.firefox.com/people/shorlander/photon/Mockups/macOS.html the
> "Pocket" icon is in the address bar. Can you please clarify where this
> should be? Thanks

It should be in the address bar.

	ovidiu boca[:Ovidiu]
	Comment 19 • 7 years ago

Thanks Dão,

I tested this issue on Ubuntu 16.04, Mac OS X 10.12, Windows 10, Windows 7 with the latest Nightly 57.0a1(2017-08-08) I can confirm the fix.