345286 - results table header row should have a class (bz_buglist_header)

Status: RESOLVED FIXED

(Bugzilla :: Query/Bug List, enhancement)

Description

[timeless]

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}

Comment 1

[timeless]

Attachment: follow on

Comment 2

[Vlad Dascalu]

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.

Comment 3

[Vlad Dascalu]

This patch is made against the patch located in the dependent bug (which is pending approval as well).

Comment 4

[timeless]

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.