Closed
Bug 1355455
Opened 8 years ago
Closed 7 years ago
Photon toolbar button icon drop
Categories
(Firefox :: Theme, enhancement, P1)
Firefox
Theme
Tracking
()
Tracking | Status | |
---|---|---|
firefox57 | --- | verified |
People
(Reporter: dao, Assigned: nhnt11)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [photon-visual][p1][57])
Attachments
(1 file)
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.
Reporter | ||
Updated•8 years ago
|
Whiteboard: [photon][57] → [photon-visual][57]
Reporter | ||
Updated•8 years ago
|
Summary: Toolbar button icon drop → Photon toolbar button icon drop
Updated•8 years ago
|
Flags: qe-verify+
Priority: P1 → P2
QA Contact: ovidiu.boca
Reporter | ||
Updated•8 years ago
|
Whiteboard: [photon-visual][57] → [photon-visual][p1][57]
Updated•8 years ago
|
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Iteration: --- → 55.4 - May 1
Priority: P2 → P1
Updated•8 years ago
|
QA Contact: ovidiu.boca → brindusa.tot
Updated•7 years ago
|
Iteration: 55.4 - May 1 → 55.5 - May 15
Reporter | ||
Updated•7 years ago
|
status-firefox55:
affected → ---
Reporter | ||
Updated•7 years ago
|
Assignee: nhnt11 → nobody
Status: ASSIGNED → NEW
Iteration: 55.5 - May 15 → ---
Priority: P1 → P2
Updated•7 years ago
|
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.
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Flags: needinfo?(nhnt11)
Priority: P2 → P1
Updated•7 years ago
|
Iteration: --- → 56.4 - Aug 1
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(nhnt11)
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
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
Attachment #8892119 -
Flags: review?(dao+bmo) → review+
Updated•7 years ago
|
Iteration: 56.4 - Aug 1 → 57.1 - Aug 15
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)
Comment hidden (mozreview-request) |
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
Backed out for failures like https://treeherder.mozilla.org/logviewer.html#?job_id=120535233&repo=autoland https://hg.mozilla.org/integration/autoland/rev/c7fda3c9ff866a9c830f8a91068af3e27a64238a
Flags: needinfo?(nhnt11)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 13•7 years ago
|
||
A whitelisted image was renamed; I updated the entry in the test file and re-landed.
Flags: needinfo?(nhnt11)
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
![]() |
||
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c9c1ce4407b9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
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)
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
Flags: needinfo?(nhnt11)
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.
Flags: needinfo?(nhnt11)
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.
You need to log in
before you can comment on or make changes to this bug.
Description
•