Closed Bug 303550 Opened 19 years ago Closed 19 years ago

Bug lists trigger standards compliance mode in Gecko, causing CLOSED bugs not to be lined through in bug lists any more

Categories

(Bugzilla :: Query/Bug List, defect)

2.20
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 2.20

People

(Reporter: Wurblzap, Assigned: Wurblzap)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

Bugzilla 2.20rc1 bug lists happily trigger standards compliance mode in
Firefox 1.0.6. This results in bugs in the CLOSED state not to be lined through
any more.

The reason consists of two effects of standards compliance mode:
 o CSS class and id names are case sensitive as per
   http://www.w3.org/TR/REC-html40/struct/global.html#h-7.5.2
 o CSS text-decoration on <tr> is not inherited to <td>
Attached patch Patch β€” β€” Splinter Review
I don't know whether the second part is a bug. This patch takes care of both.

We still need bz_closed because GetBugLink in globals.pl issues this class for
bugs in non-open states (not only for the CLOSED state).
Attachment #191685 - Flags: review?
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.20
This still works in 2.18 (see
http://landfill.bugzilla.org/bugzilla-2.18-branch/buglist.cgi?bug_status=CLOSED).
Keywords: regression
Comment on attachment 191685 [details] [diff] [review]
Patch

This fixes the problem for me: r=LpSolit for both 2.20 and tip.

Marc, can you explain me why 2.18.3 is not affected? By looking at the source
code of a buglist page, it looks really similar the one obtained from 2.20+.
Attachment #191685 - Flags: review? → review+
(In reply to comment #1)
> We still need bz_closed because GetBugLink in globals.pl issues this class for
> bugs in non-open states (not only for the CLOSED state).

GetBugLink is not the only one to use bz_closed, see for instance "FILTER
closed()" in Templates.pm.
Flags: approval?
Flags: approval2.20?
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
tip:

Checking in skins/standard/global.css;
/cvsroot/mozilla/webtools/bugzilla/skins/standard/global.css,v  <--  global.css
new revision: 1.14; previous revision: 1.13
done

2.20rc2:

Checking in skins/standard/global.css;
/cvsroot/mozilla/webtools/bugzilla/skins/standard/global.css,v  <--  global.css
new revision: 1.12.4.1; previous revision: 1.12
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: