Dont show the download icon in the start-download animation when the button is currently hidden in the toolbar.
Categories
(Firefox :: Theme, enhancement, P2)
Tracking
()
People
(Reporter: sfoster, Assigned: sfoster)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-icons] [priority:2a] [proton-uplift])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
The default behavior is to hide the download toolbar button and only show it in the toolbar when a download happens or there are contents in the download panel.
The new download start animation includes the icon in the first few frames, so it appears to flash in and them disappear quickly as it transitions to the progress circle.
In the initial download case, we should remove the icon from those first few frames.
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
I've filed bug 1709904 and bug 1709902 to follow up on the issues Gijs identified in review.
Comment 4•4 years ago
|
||
bugherder |
Comment 5•4 years ago
|
||
Request for Uplift from the PM team
Updated•3 years ago
|
Updated•3 years ago
|
Comment 6•3 years ago
|
||
Comment on attachment 9220243 [details]
Bug 1709172 - Add a washidden attribute on #downloads-button; adjust fill color in start animation based on state we are animating from. r?Gijs
Beta/Release Uplift Approval Request
- User impact if declined: Flickery download animation for the first download in every session
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: n/a
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Relatively small changes to JS, CSS and image assets, has baked on nightly for a decent while now, applies completely cleanly to beta branch.
- String changes made/needed: Nope
Updated•3 years ago
|
Comment 7•3 years ago
|
||
Comment on attachment 9220243 [details]
Bug 1709172 - Add a washidden attribute on #downloads-button; adjust fill color in start animation based on state we are animating from. r?Gijs
Approved for 89 beta 15, thanks.
Comment 8•3 years ago
|
||
bugherder uplift |
Description
•