401789 - Tables don't have good identifiers for use with css

Status: RESOLVED WORKSFORME

(Bugzilla :: User Interface, defect, P3)

Description

[Pablo Bender]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; pt-BR; rv:1.8.1.8) Gecko/20071008 Firefox/2.0.0.8
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; pt-BR; rv:1.8.1.8) Gecko/20071008 Firefox/2.0.0.8

I have made some changes to the CSS of Bugzilla on skin/custom folder and I can't override the color #cccccc of the header of TimeTracking table on the show_bug.cgi (edit.html.tmpl), for example.

Of course, I can do the change by editing the edit.html.tmpl file, but I can't do it just changing the css.

This table does not have any css identifier to do style modifications on it, and have static style properties like cellspacing, padding, border and background color on edit.html.tmpl).

If a css is applied on "table th", it will affect others cells, like "Status", "Resolution" on the same page.

On the other hand, the Attachment table have id=attachment_table, that allows override your style.

Reproducible: Always

Steps to Reproduce:
Edit the global.css file to try overrides the background color of the TimeTracking table
Actual Results:  
It's impossible to override style of TimeTracking table without customize edit.html.tmpl template

Comment 1

[Frédéric Buclin]

Let's try to fix this bug on time for Bugzilla 3.2. Pablo, are you interested in working on it?

Comment 2

[Emmanuel Seyman]

(In reply to comment #1)
>
> Let's try to fix this bug on time for Bugzilla 3.2.

I'ld be glad to lend a hand on this one.

Comment 3

[Pablo Bender]

(In reply to comment #1)
> Let's try to fix this bug on time for Bugzilla 3.2. Pablo, are you interested
> in working on it?
> 

Yes, I am... But, how can I help? I have to register somewhere?
(... Sorry for posting questions here)

Comment 4

[Emmanuel Seyman]

(In reply to comment #3)
>
> Yes, I am... But, how can I help? I have to register somewhere?

All Bugzilla development is done in bugzilla.mozilla.org so you
registered when you created your account here.

We have a developer's guide which you can read through but the 30 second
version is :

- you create a patch which changes the table definition for the timetracking
data from :
<table cellspacing="0" cellpadding="4" border="1">

to :
<table id="timetracking" cellspacing="0" cellpadding="4" border="1">

- you ask for review on this patch
- once review is granted, somebody approves the bug
- once that is done, your patch is committed to Bugzilla's CVS.

Comment 5

[Frédéric Buclin]

Bugzilla 3.2 is restricted to security bugs only. Moreover, this bug is either assigned to nobody or got no traction for several months now. Rather than retargetting it at each new release, I'm clearing the target milestone and the bug will be retargetted to some sensible release when someone starts fixing this bug for real (Bugzilla 3.8 more likely).

Comment 6

[Sunil Joshi [:sjoshi]]

Now we have css class for timetracking table named "bz_time_tracking_table".
This is WFM.

Comment 7

[Frédéric Buclin]

Yes, WFM since at least Bugzilla 3.2.