962636 - Progress meters need to be updated for Windows 8/8.1

Status: RESOLVED DUPLICATE of bug 642846

(Core :: Widget: Win32, defect)

Description

[ebrahim]

Attachment: FirefoxNightly29VSOperaNext19.png

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release)
Build ID: 20140122030521

Steps to reproduce:

Open data:text/html,<p><progress /></p><p><progress max=100 value=100 /></p> on Firefox and Opera Next


Actual results:

They differs and Opera Next looks highly more better


Expected results:

* Filled progress and the not filled one shouldn't look same and stall progress should be empty.
and IMO:
* A native look progress should not have padding inside on Windows 8
* It should not have radius on Windows 8

Comment 1

[Sidhant Sharma [:TigerKid001]]

Hi! I'd like to work on this bug. Can i get a hint where to start?

Comment 2

[Jim Mathies [:jimm]]

Looks like this may be a bug on all platforms based on the test case.

data:text/html,<p><progress/><p><progress max=100 value=100/><p><progress max=100 value=10/>

the first progress is filled, and indeterminate. I wonder if that's correct?

http://www.w3.org/TR/html5-author/the-progress-element.html

Comment 3

[Jim Mathies [:jimm]]

Sidhant, I'd suggest starting by figuring out what the exact bug is based on the original report. Then we can figure out which area of the code base is involved with the bug.

Comment 4

[ebrahim]

The bug was the indeterminate progress look should be a unfilled progressbar with an additional wave on it not a filled one and also wave.

Comment 5

[Sidhant Sharma [:TigerKid001]]

(In reply to ebrahim from comment #4)
> The bug was the indeterminate progress look should be a unfilled progressbar
> with an additional wave on it not a filled one and also wave.
Yes, I have reproduced this bug, and understand that the progress meter meter should only be partially filled, not fully filled. I would like to know where should I start looking so I can crunch this bug soon :)

Comment 6

[Jim Mathies [:jimm]]

Sorry, this is a dupe.

Comment 7

[ebrahim]

One day someone ask to fill a separate bug https://bugzilla.mozilla.org/show_bug.cgi?id=642846#c32 and one day someone mark it as a dupe on it again, interesting :)

Comment 8

[Jim Mathies [:jimm]]

(In reply to ebrahim from comment #7)
> One day someone ask to fill a separate bug
> https://bugzilla.mozilla.org/show_bug.cgi?id=642846#c32 and one day someone
> mark it as a dupe on it again, interesting :)

Yeah I noticed that too, sorry. Having worked with win8 now for a bit I'm confident this is a dupe, although we may need to add a new graphic.