results table header row should have a class (bz_buglist_header)

RESOLVED FIXED in Bugzilla 3.2

Status

()

Bugzilla
Query/Bug List
--
enhancement
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

2.21
Bugzilla 3.2
Bug Flags:
approval +

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
If you have a long query, you will get something like:
               ID  	 Sev  	 Pri  	 Plt  	 Assignee  	 Reporter  	 Status  	 Resolution  	 Product  	 Summary
<bugs...>
               ID  	 Sev  	 Pri  	 Plt  	 Assignee  	 Reporter  	 Status  	 Resolution  	 Product  	 Summary
<morebugs...>
               ID  	 Sev  	 Pri  	 Plt  	 Assignee  	 Reporter  	 Status  	 Resolution  	 Product  	 Summary
<morebugs...>

Steps to reproduce:
If you really want such a query, search for all timeless's bugs or all bugs filed by timeless, or all open bugs filed by timeless or maybe just all open bugs filed by timeless in product bugzilla.
https://bugzilla.mozilla.org/buglist.cgi?bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailreporter1=1&emailtype1=exact&email1=timeless%40bemail.org&field0-0-0=bug_status&type0-0-0=notequals&value0-0-0=UNCONFIRMED&field0-0-1=reporter&type0-0-1=equals&value0-0-1=timeless%40bemail.org
https://bugzilla.mozilla.org/buglist.cgi?&emailreporter1=1&emailtype1=exact&email1=timeless%40bemail.org
https://bugzilla.mozilla.org/buglist.cgi?&emailreporter1=1&emailtype1=exact&email1=timeless%40bemail.org&resolution=---
https://bugzilla.mozilla.org/buglist.cgi?&emailreporter1=1&emailtype1=exact&email1=timeless%40bemail.org&resolution=---&product=Bugzilla

That last one really does give me three headers.

Expected results (from working with someone who just learned about userContent.css):
the first header should have a special class. It's true that we were able to work around this problem by using table+table>tr>th{display:none}, but that's really annoying, it should be possible to just do:

tr.bz_buglist_header{visibility:collapsed}
tr.bz_first_buglist_header{visibility:default}
(Assignee)

Updated

12 years ago
Keywords: ue
(Assignee)

Comment 1

11 years ago
Created attachment 276392 [details] [diff] [review]
follow on
Assignee: query-and-buglist → timeless
Status: NEW → ASSIGNED
Attachment #276392 - Flags: review?(vladd)

Updated

11 years ago
OS: Windows XP → All
Hardware: PC → All

Comment 2

11 years ago
Comment on attachment 276392 [details] [diff] [review]
follow on

The "collapse additional table headers" thing should be some kind of prefs, it's really not a thing for CSS, but oh well.

Ignoring it, I think there still are useful cases where end-users might want to have the first buglist header rendered differently, so this makes sense to me.
Attachment #276392 - Flags: review?(vladd) → review+

Comment 3

11 years ago
This patch is made against the patch located in the dependent bug (which is pending approval as well).
Flags: approval?
Target Milestone: --- → Bugzilla 3.2

Updated

11 years ago
Flags: approval? → approval+
Keywords: ue
(Assignee)

Comment 4

11 years ago
Comment on attachment 276392 [details] [diff] [review]
follow on

mozilla/webtools/bugzilla/template/en/default/list/table.html.tmpl 	1.36

comment 3, yeah um, oops. I botched this one. I committed the other patch. Well, kinda fixable, swapping summaries here.
Blocks: 345285
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
No longer depends on: 345285
Resolution: --- → FIXED
Summary: the first query results table header should have an extra class (bz_first_buglist_header) → results table header row should have a class (bz_buglist_header)
You need to log in before you can comment on or make changes to this bug.