Claro Theme Grid.css .dojoxGridHeader .dojoxGridCell has padding issue.

[Kitson Kelly]

There is a padding issue in the header of claro themed Grids, where the background gradient icon is padded out leaving 3px wide of non gradient and making the header look "ugly". It appears there is a 3px padding that is in there which can be removed by adding 0 padding to .claro .dojoxGridHeader .dojoxGridCell which the attached patch does.

My CCLA has been on file from 28 April with Aimee.

[Kitson Kelly]

Adjusts the header padding for the claro theme

[bill]

Evan, seem OK to you?

[Kitson Kelly]

I found an issue with my patch, because the rows calculate considering the padding in the header, the row widths become mis-aligned.

Basically I am not sure what the header is supposed to look like in the claro theme, but it currently looks "ugly" and I don't think it is what is intended. The padding gets worse when you have a multi-line header. I get the same problem with FF 3.6 and Chrome 5, which I have attached a screen grab.

[Kitson Kelly]

Example of a grid header padding problem.

[evan]

Not seeing the 3px padding issue in latest ​trunk, will upload a patch to fix the multiple line issue in Claro.

BTW, wasn't able to see the "grid issue.tiff" snapshot.

[Adam Peller]

(In [22438]) Patch from Evan: prevent content from getting wrapped in header cell. Refs #11104 !strict

[Adam Peller]

please reopen if you can still demonstrate the 3px padding issue

[evan]

(In [23794]) Partially roll back [22438] - column headers should be wrapped by default, only prohibit wrapping when NestedSorting? is on, refs #11104, !strict