Patch #26674

Additional css for query column headers

Added by Dmitry Lisichkin about 1 month ago. Updated about 1 month ago.

Status:Needs feedbackStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:UI
Target version:Candidate for next major release

Description

For now there no any chances to style table headers for several columns becouse there no any css classes (it just <th> with column caption)
In patch I offer to add two classes: query-column-header and query-column-header-#{column.name}

additional_css.patch Magnifier (733 Bytes) Dmitry Lisichkin, 2017-08-14 11:38

History

#1 Updated by Go MAEDA about 1 month ago

  • Status changed from New to Needs feedback
  • Target version set to Candidate for next major release

I agree that adding classes is a nice improvement for theme developers. But I think that class names such as "query-column-header query-column-header-issue" is long and "query-column-header issue" is enough.

What do you think?

#2 Updated by Dmitry Lisichkin about 1 month ago

Go MAEDA wrote:

I agree that adding classes is a nice improvement for theme developers. But I think that class names such as "query-column-header query-column-header-issue" is long and "query-column-header issue" is enough.

What do you think?

Class name contained only column name is not enough to specify context. It may be used in another parts of application and its style will be attached to query-column-header too. For example: if we have column with name 'highlight' then column header will be with background (.highlight { background-color: #FCFD8D;} from application.css).

Also available in: Atom PDF