185475 - Progressmeter is invisible when mode="undetermined" in Classic Theme

Status: RESOLVED WORKSFORME

(Core Graveyard :: Skinability, defect)

Description

[Matthew Wilson]

User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.3a) Gecko/20021212
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.3a) Gecko/20021212

In Mozilla 1.3a (but not in 1.2.1), a progressmeter with mode="undetermined"
does not display anything in the Classic Theme.

Reproducible: Always

Steps to Reproduce:
1. Load a document with a progressmeter with mode="undetermined" in the Classic
Theme.

Actual Results:  
No element was visible.

Expected Results:  
A scrolling progress meter should be displayed.

Comment 1

[Matthew Wilson]

Attachment: XUL demonstration file containing a progressmeter

Comment 2

[tloehnert]

hope this is related, if not I apologize; it also applies to the classic theme:
after upgrading from 1.2.1 to 1.3a (Mozilla/5.0 (Windows; U; Windows NT 5.0;
en-US; rv:1.3a) Gecko/20021216) I discovered that the "scrolling progressmeter"
that used to show up if Mailnews was sending the login/password to the
Mailserver  is missing.
Furthermore: the standard progressmeter (e.g. while loading a webpage) has lost
its discreet gray color and turned dark blue. Also: the standard progressmeter
in the download-progress dialog has lost its "boundaries".

Comment 3

[Matthias Bockelkamp]

Assigning to me.

Comment 4

[Matthias Bockelkamp]

Attachment: Patch

Comment 5

[Matthias Bockelkamp]

Attachment: Patch v2

I just saw this is also necessary for Macintosh (but I cannot test it).

Comment 6

[Christian :Biesinger (don't email me, ping me on IRC)]

Comment on attachment 131427 [details] [diff] [review]
Patch v2

please don't ask me for review on general ui bugs. try neil.

that said, can you explain why this fixes anything? Are you sure your patch
causes no CSS syntax error?

Comment 7

[[e-mail]]

Comment on attachment 131427 [details] [diff] [review]
Patch v2

>RCS file: /cvsroot/mozilla/themes/classic/global/win/progressmeter.css,v
>-  -moz-appearance: progressbar;
This styles is needed for Windows XP theme support for normal progressmeters.
Unfortunately the appearance overrides the background image, which is
unexpected.

Comment 8

[Matthias Bockelkamp]

Note to self: Caused by Bug 115748

Comment 9

[Scott Gammans]

The "Smoke" theme also doesn't work for undetermined progress meters... I just
tried Alice Corbin's mega XUL test page
(http://www.hevanet.com/acorbin/xul/top.xul) and the undetermined progress meter
ain't there.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a) Gecko/20040428

Comment 10

[Matthias Bockelkamp]

There is a working progressmeter for mode="undetermined" for some time now.