280673 - Background color of table headers should be unified (patch / change log enclosed!)

Status: RESOLVED DUPLICATE of bug 274658

(Bugzilla :: User Interface, defect)

Description

[Peter Bittner]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; de-DE; rv:1.7.5) Gecko/20041122 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; de-DE; rv:1.7.5) Gecko/20041122 Firefox/1.0

Tables with headers are not colored in the same way and not even with the same
color. Some table headers are (kind of) blue, some just white which is not very
helpful for the user to identify objects that serve the same or similar purpose.

I have done some customizing on that fact:

 - removed the "bgcolor=..." attributes from tr- and th- tags
 - added "class=..." attributes to all tr- tags that form a table header
 - added a corresponding style for tr and th to web/css/global.css
 - and some minor beautifying

Changes apply to some templates and some of the .cgi scripts as well.

Please consider for a future release, the changes would improve customizability.
Cheers, Peter


Reproducible: Always

Steps to Reproduce:
A. Go to "Edit User Preferences" / "Saved Searches" , or
B. Go to "Edit User Preferences" / "Email settings" , or
C. Go to "Edit Keywords", or
D. Go to "Edit Groups", or
E. Go to "Edit Flags", or
F. Go to "Edit Users" / "Select User", or
G. Go to "Edit "
Actual Results:  
Some table haeaders blue, some white.

Expected Results:  
After applying the changes according to the attached change log all table
headers appear in the same background color. The respective color is specified
in web/css/global.css then.

I am using version 2.18 of Bugzilla on an Debian Sarge installation.

Notation in the attached change log is - hopefully - obvious:
(...filename...)  ...  means: the following changes apply to file 'filename'
<  ...  means: code removed
>  ...  means: code added (instead of, if applicable)

Comment 1

[Peter Bittner]

Attachment: Attached file: Change Log for customization (based on version 2.18 of Bugzilla)

Comment 2

[Marc Schumann [:Wurblzap]]

I like the idea.
Do you know how to produce a patch file? This would make the testing and
reviewing process a whole lot easier :)

Comment 3

[Peter Bittner]

(In reply to comment #2)
> I like the idea.
> Do you know how to produce a patch file? This would make the testing and
> reviewing process a whole lot easier :)

I thought a patch wouldn't help for the CVS version. (Anyway, if I do it I need
to come back to you later as I need to get our Bugzilla customization ready
before the deadline of our project! *streeeeeeeess*)  8-(

Hope my change log is helpful to anyone anyway...

Comment 4

[Frédéric Buclin]

If you have a patch, please submit it in bug 274658. :)


*** This bug has been marked as a duplicate of 274658 ***