Opened 12 years ago

Closed 9 years ago

#11186 closed defect (patchwelcome)

Dojox.grid.DataGrid does work properly for hidden column after resizing column

Reported by: jainpramod1983 Owned by: Bryan Forbes
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.5.0b2
Keywords: Cc:
Blocked By: Blocking:

Description

If you have hidden columns in a grid that has column's width defined in percentage. After resizing a column, while un-hiding the hidden column, column does not becomes visible. The problem is with convertColPctToFixed method in dojox.grid._View.js file where while converting the width to pixels, it does not sets the correct width to hidden columns.

Attachments (4)

djtestcase.html​ (3.1 KB) - added by jainpramod1983 12 years ago.
attachment part-1 for testing the defect
movies.csv​ (324 bytes) - added by jainpramod1983 12 years ago.
attachment part-2 for testing the defect (Data Store)
djtestcase_withPatch.html​ (3.6 KB) - added by jainpramod1983 12 years ago.
Patch to fix the issue - Modified convertColPctToFixed method to fix the error
11186_gridPctHiddenCol.patch​ (1.6 KB) - added by Nathan Toone 12 years ago.
Unified diff patch from the attached HTML file. Note, this patch is based off the 1.4.3 codebase...and has not been extensively tested.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 12 years ago by bill

Component: General → DojoX Grid
Owner: changed from anonymous to bryanforbes

Changed 12 years ago by jainpramod1983

Attachment: djtestcase.html​ added

attachment part-1 for testing the defect

Changed 12 years ago by jainpramod1983

Attachment: movies.csv​ added

attachment part-2 for testing the defect (Data Store)

Changed 12 years ago by jainpramod1983

Attachment: djtestcase_withPatch.html​ added

Patch to fix the issue - Modified convertColPctToFixed method to fix the error

Changed 12 years ago by Nathan Toone

Attachment: 11186_gridPctHiddenCol.patch​ added

Unified diff patch from the attached HTML file. Note, this patch is based off the 1.4.3 codebase...and has not been extensively tested.

comment:2 Changed 12 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:3 Changed 9 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:4 Changed 9 years ago by bill

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