Closed Bug 1392793 Opened 7 years ago Closed 7 years ago

Replace reload/stop assets to account for transparency

Categories

(Firefox :: General, defect, P1)

57 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 57
Iteration:
57.3 - Sep 19
Tracking Status
firefox57 --- verified

People

(Reporter: epang, Assigned: jaws)

References

Details

(Whiteboard: [reserve-photon-animation])

Attachments

(2 files)

Attached file SVGs + JSONs.zip
Replace the svg sprite asset so it works with transparency.
Flags: qe-verify-
Flags: qe-verify- → qe-verify+
Priority: -- → P4
QA Contact: stefan.georgiev
Summary: [ux] Replace refresh/stop assets to account for transparency → Replace refresh/stop assets to account for transparency
Whiteboard: [photon-animation] [triage] [ux] → [reserve-photon-animation]
Priority: P4 → P3
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Iteration: --- → 57.3 - Sep 19
Priority: P3 → P1
Summary: Replace refresh/stop assets to account for transparency → Replace reload/stop assets to account for transparency
Comment on attachment 8902936 [details]
Bug 1392793 - Replace reload/stop assets to account for transparency.

https://reviewboard.mozilla.org/r/174670/#review179850

Thanks! The SVG are better now, but I did find stuff to remove. :) The cleaned SVG should take in account the opacity, but please let me know if they don't.

r+ with the cleaned SVG used.

::: browser/themes/shared/compacttheme.inc.css:54
(Diff revision 1)
>    --chrome-selection-color: #f5f7fa;
>    --chrome-selection-background-color: #4c9ed9;
>  }
>  
>  .toolbarbutton-animatable-box[brighttext],
> +toolbar[brighttext] .toolbarbutton-animatable-box,

Good catch! I don't know why I haven't noticed this issue until now (even though I use the dark theme daily).

::: browser/themes/shared/icons/reload-to-stop.svg:5
(Diff revision 1)
>  <!-- This Source Code Form is subject to the terms of the Mozilla Public
>     - License, v. 2.0. If a copy of the MPL was not distributed with this
>     - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
>  <svg xmlns="http://www.w3.org/2000/svg" width="468" height="20" fill="context-fill">
> -  <svg x="0">
> +  <svg>

I've optimized this SVG: https://gist.github.com/nt1m/663b147ec798964fb4089d3bf8faf0c9

Many masks with no visible effects have been removed, some groups have been flattened.

::: browser/themes/shared/icons/stop-to-reload.svg:4
(Diff revision 1)
>  <!-- This Source Code Form is subject to the terms of the Mozilla Public
>     - License, v. 2.0. If a copy of the MPL was not distributed with this
>     - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
>  <svg xmlns="http://www.w3.org/2000/svg" width="468" height="20" fill="context-fill">

I've optimized this SVG as well: https://gist.github.com/nt1m/663b147ec798964fb4089d3bf8faf0c9#file-stop-to-reload-svg
Attachment #8902936 - Flags: review?(ntim.bugs) → review+
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8676b5208a5
Replace reload/stop assets to account for transparency. r=ntim
https://hg.mozilla.org/mozilla-central/rev/d8676b5208a5
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 (20170915100121)

This issue is verified as fixed with the latest Nightly build on 9/15/2017.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: