Closed Bug 616641 Opened 9 years ago Closed 9 years ago

Start/end caps of the download-progress binding broken on Windows/OSX with undetermined mode

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
mozilla2.0b8
Tracking Status
blocking2.0 --- final+

People

(Reporter: whimboo, Assigned: Unfocused)

References

()

Details

(Keywords: polish, Whiteboard: [AOMTestday])

Attachments

(3 files)

Attached image screenshot
Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101203 Firefox/4.0b8pre

The undetermined download-progress has a graphical glitch for both start- and end-caps. While the start-cap is always green, the end-cap doesn't receive the green progress. See the screenshot.

Steps:
1. Open the link from the URL field
Updated steps:

1. Copy the URL from the field above
2. Open the Add-ons Manager
3. Paste the URL into the location bar and hit return.
Same happens on OS X, but not with the native theme on Linux.
Summary: Start/end caps of the download-progress binding broken on Windows → Start/end caps of the download-progress binding broken on Windows/OSX with undetermined mode
Attached patch Patch v1Splinter Review
This patch makes it so when the progress meter is in undetermined mode, both end caps are filled, and the colored border extends all the way around the widget (looks a lot better that way, due to both caps being filled). Also moves the buttons into the center by 2px, to make a gap between them and the end caps.

Also fixes bug 616643. Apparently, OSX has never had an undetermined mode (without manually attaching the binding, that is) - so this attaches the undetermined binding in pinstripe.

The download progress bar on Linux doesn't have end caps to round the ends, so never had this issue.
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Attachment #495450 - Flags: review?(dtownsend)
Attachment #495450 - Flags: review?(dtownsend) → review+
blocking2.0: --- → final+
https://hg.mozilla.org/mozilla-central/rev/63fc6076169b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite-
Flags: in-litmus?
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
Verified fixed with Mozilla/5.0 (Windows NT 5.1; rv:2.0b8pre) Gecko/20101207 Firefox/4.0b8pre

The Litmus test is covered by bug 553479.
Status: RESOLVED → VERIFIED
Flags: in-litmus? → in-litmus-
You need to log in before you can comment on or make changes to this bug.