Opened 12 years ago

Closed 11 years ago

#11516 closed defect (fixed)

claroGrid.css missing styles for treeGrid

Reported by: davidqge Owned by: Bryan Forbes
Priority: high Milestone: 1.5.1
Component: DojoX Grid Version: 1.5
Keywords: treeGrid Claro Cc:
Blocked By: Blocking:


The test_treegrid.html didn't work after changed tundra to claro theme.

Here is the patch too add the styles to claroGrid.css, so test_treegrid.html would work with claro theme.

The values are taken from existing claro/tree.css and tundraGrid.css.

Attachments (1)

claro-treegrid.patch (1.5 KB) - added by davidqge 12 years ago.

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by davidqge

Attachment: claro-treegrid.patch added

comment:1 Changed 12 years ago by davidqge

Patch is from David Ge

comment:2 Changed 12 years ago by bill

Hi David, please file a CLA so that we can use your patch (and add a comment here when you've filed it), thanks!

Also, the other thing is that we try not to use !important in theme files as it makes life difficult for people trying to override the theme. Can you avoid that usage?

comment:3 Changed 11 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:4 Changed 11 years ago by Bryan Forbes

(In [23325]) Fixed TreeGrid? styles for claro. refs #11516

comment:5 Changed 11 years ago by Bryan Forbes

Milestone: tbd1.5.1
Resolution: fixed
Status: newclosed

I fixed this by copying and modifying the tundra styles and I backported it to the 1.5 branch. The patch was not used.

Note: See TracTickets for help on using tickets.