Opened 9 years ago

Closed 6 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 9 years ago.
attachment part-1 for testing the defect
movies.csv​ (324 bytes) - added by jainpramod1983 9 years ago.
attachment part-2 for testing the defect (Data Store)
djtestcase_withPatch.html​ (3.6 KB) - added by jainpramod1983 9 years ago.
Patch to fix the issue - Modified convertColPctToFixed method to fix the error
11186_gridPctHiddenCol.patch​ (1.6 KB) - added by Nathan Toone 9 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 9 years ago by bill

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

Changed 9 years ago by jainpramod1983

Attachment: djtestcase.html​ added

attachment part-1 for testing the defect

Changed 9 years ago by jainpramod1983

Attachment: movies.csv​ added

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

Changed 9 years ago by jainpramod1983

Attachment: djtestcase_withPatch.html​ added

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

Changed 9 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 9 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

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

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