Opened 10 years ago

Closed 8 years ago

#12118 closed defect (patchwelcome)

dojox.grid.TreeGrid, rowHeight integer value aren't used in sub level rows

Reported by: harobed Owned by: Evan
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.5
Keywords: TreeGrid rowHeight Cc: treegrid, rowheight
Blocked By: Blocking:

Description

Hi,

in dojox.grid.TreeGrid? when I set rowHeight property to integer value, example 40 then rows at first level are set with 40px height value but all sub level rows height are in auto mode.

Regards, Stephane

Attachments (3)

treegrid1.png​ (3.1 KB) - added by harobed 10 years ago.
First screen shot with only first level rows
treegrid2.png​ (4.5 KB) - added by harobed 10 years ago.
Second screenshot with sub row in auto height mode
0001-Fix-12118-issue-dojox.grid.TreeGrid-rowHeight-intege.patch​ (6.3 KB) - added by harobed 10 years ago.
This a fix patch

Download all attachments as: .zip

Change History (10)

Changed 10 years ago by harobed

Attachment: treegrid1.png​ added

First screen shot with only first level rows

Changed 10 years ago by harobed

Attachment: treegrid2.png​ added

Second screenshot with sub row in auto height mode

Changed 10 years ago by harobed

Attachment: 0001-Fix-12118-issue-dojox.grid.TreeGrid-rowHeight-intege.patch​ added

This a fix patch

comment:1 Changed 10 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:2 Changed 10 years ago by bill

Milestone: 1.6 → 1.7

punting new-ish 1.6 tickets to 1.7

comment:3 Changed 10 years ago by evan

Owner: changed from Bryan Forbes to evan
Priority: normal → high

comment:4 Changed 10 years ago by evan

Cc: treegrid rowheight added
Milestone: 1.7 → tbd
Priority: high → normal

Won't be able to get this in for 1.7, a through work will be needed since rowHeight is also not working well for base DataGrid?

comment:5 Changed 10 years ago by Evan

Owner: changed from evan to Evan

comment:6 Changed 8 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:7 Changed 8 years ago by bill

Resolution: → patchwelcome
Status: new → closed
Note: See TracTickets for help on using tickets.