Closed Bug 642127 Opened 15 years ago Closed 14 years ago

Improve the Windows Vista/7 progress bar glow effect

Categories

(Core :: Widget: Win32, defect)

Product:
Core
Component:
Widget: Win32
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla6

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(2 files, 1 obsolete file)

Patch v1.1
3.24 KB, patch
jimm
: review+
Details | Diff | Splinter Review
Download Window (green bar does not use entire height in 5/11/11 Nightly)
101.67 KB, image/png
Details
For the moment, the progress bar glow effect is taking into account the bar size but should take the entire progress bar size. It would make the effect less speedy and stressful ;)
Attached patch Patch v1 (obsolete) — — Splinter Review
By tweaking the speed and adding a delay we seem to be close to what is usually done on Windows 7 (didn't check Vista but I assume it's not so far). Though, the speed and delay seems to vary between applications. I guess it's customizable.
Assignee: nobody → mounir.lamouri
Status: NEW → ASSIGNED
Attachment #519938 - Flags: review?(jmathies)
Whiteboard: [needs review]
(In reply to comment #0) > For the moment, the progress bar glow effect is taking into account the bar > size but should take the entire progress bar size. It would make the effect > less speedy and stressful ;) By the way, this comment is wrong. This is what Chrome is doing but that's not what Windows native widget does...
Attached patch Patch v1.1 — — Splinter Review
With constants.
Attachment #519938 - Attachment is obsolete: true
Attachment #519938 - Flags: review?(jmathies)
Attachment #519976 - Flags: review?(jmathies)
Comment on attachment 519976 [details] [diff] [review] Patch v1.1 Review of attachment 519976 [details] [diff] [review]: r+
Attachment #519976 - Flags: review?(jmathies) → review+
Whiteboard: [needs review]
Whiteboard: [ready to land][waits for dependencies]
Pushed: http://hg.mozilla.org/mozilla-central/rev/f8ab655d503c http://hg.mozilla.org/mozilla-central/rev/4a7b8019b83f
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [ready to land][waits for dependencies]
Target Milestone: --- → mozilla6
Depends on: 655860
Backed out in http://hg.mozilla.org/mozilla-central/rev/dd9ba28d2bd9 to resolve bug 655860.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
The regression wasn't caused by these patches. Re-landed: http://hg.mozilla.org/mozilla-central/rev/c15b58d50e45
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
No longer depends on: 655860
This broke something. Now, when you download a file, the filled in green area does not take up the entire height of the bar and looks weird. Screenshot provided.
(In reply to comment #9) > This broke something. Now, when you download a file, the filled in green > area does not take up the entire height of the bar and looks weird. > Screenshot provided. I am using Build ID = http://hg.mozilla.org/mozilla-central/rev/a45b5cf1d625
(In reply to comment #9) > This broke something. Now, when you download a file, the filled in green > area does not take up the entire height of the bar and looks weird. > Screenshot provided. filed bug 656284.
Mozilla/5.0 (X11; Linux i686; rv:6.0) Gecko/20100101 Firefox/6.0 Everything seems in order now and bug 656284 got also fixed. Setting status to Verified Fixed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: