555955 - Bugzilla CSS throws lots of warnings on Mozilla error console

Status: RESOLVED INVALID

(Bugzilla :: User Interface, defect)

Description

[Ben Bucksch (:BenB)]

Reproduction:
- Start Firefox (trunk)
- Open error console
- Load any bugzilla.mozilla.org page (show_bug, commit, you name it)

Actual result:
See about a dozen CSS warnings fly by.

Warning: Error in parsing value for 'white-space'.  Declaration dropped.
Source File: https://bugzilla.mozilla.org/skins/standard/global.css
Line: 278 (2x, plus 1x line 280)
Warning: Expected declaration but found '*'.  Skipped to next declaration.
Source File: https://bugzilla.mozilla.org/skins/standard/yui/calendar.css
Line: 7 (5x)
Warning: Unknown property 'zoom'.  Declaration dropped.
Source File: https://bugzilla.mozilla.org/skins/standard/yui/calendar.css
Line: 7
Warning: Unknown property '-moz-opacity'.  Declaration dropped.
Source File: https://bugzilla.mozilla.org/skins/standard/yui/autocomplete.css
Line: 7
Warning: Error in parsing value for 'filter'.  Declaration dropped.
Source File: https://bugzilla.mozilla.org/skins/standard/yui/autocomplete.css
Line: 7


Expected result:
No warnings.

Comment 1

[Reed Loden [:reed]]

(In reply to comment #0)
> Warning: Unknown property '-moz-opacity'.  Declaration dropped.
> Source File: https://bugzilla.mozilla.org/skins/standard/yui/autocomplete.css
> Line: 7

Indeed, it should be just "opacity".

Comment 2

[Max Kanat-Alexander]

  Mostly, those are issues in YUI that allow backwards-compatibility. They have to do with YUI, and if you think they're actual issues upon investigation, you can report them to the YUI team.

  The white-space issue is cross-browser compatibility, it has no functional bad effect.

Comment 3

[Ben Bucksch (:BenB)]

Sorry, the bug is valid as filed.

- reed said the -moz-opacity (which definitely is for Mozilla) should be just opacity.
- The white-space: -moz-pre-wrap et al is for .bz_comment_text, and that is a <pre> *anyways*:<pre class="bz_comment_text"...
So you can remove that entirely
- "Expected declaration but found '*'" sounds like bad grammar.

Please don't just close bugs without even looking at it properly, thank you.

Comment 4

[Max Kanat-Alexander]

(In reply to comment #3)
> - reed said the -moz-opacity (which definitely is for Mozilla) should be just
> opacity.

  That's in YUI.

> - The white-space: -moz-pre-wrap et al is for .bz_comment_text, and that is a
> <pre> *anyways*:<pre class="bz_comment_text"...
> So you can remove that entirely

  No, we can't. That's pre-wrap, not pre. That's cross-browser support, there.

> - "Expected declaration but found '*'" sounds like bad grammar.

  That's YUI.

> Please don't just close bugs without even looking at it properly, thank you.

  I did, I read each message you posted.

  The YUI parts are INVALID, the pre-wrap parts are WONTFIX.

Comment 5

[Ben Bucksch (:BenB)]

> That's in YUI.

What's that supposed to tell me? YUI is part of bugzilla, therefore your bug.

Your treatment of bug reporters is just flat out rude.

Comment 6

[Reed Loden [:reed]]

Is it because the version of YUI used by Bugzilla is so old, or what exactly?