Opened 8 years ago

Closed 6 years ago

#14102 closed defect (patchwelcome)

dojox.grid._Layout problem on header's colSpan using JsonRestStore

Reported by: cranespud Owned by: Evan
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.6.1
Keywords: Cc:
Blocked By: Blocking:

Description

Hi, seems that I found what could a bug on dojox.grid._Layout I'm trying to use colSpan on the first row of a header and it causes the columns to dissapear (on grid header and grid view) as width is empty.

This happens only when using dojox.data.JsonRestStore?, and only when colSpan is used on the first row. I tested creating the layout programmatically and from markup, having the same result

I solution to change w = 0 by w = 'auto' on method addCellDef in dojox/grid/_Layout.js

addCellDef: function(inRowIndex, inCellIndex, inDef){

var self = this; var getCellWidth = function(inDef){

var w = 0; if(inDef.colSpan > 1){

w = 'auto'; fix to colSpan on header 1st row

}else{

Attached is a couple of files to prove test this Thanks and keep up with the good work!

Attachments (2)

test3.html (1.9 KB) - added by cranespud 8 years ago.
json_table.php (1.2 KB) - added by cranespud 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by cranespud

Attachment: test3.html added

Changed 8 years ago by cranespud

Attachment: json_table.php added

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

Resolution: patchwelcome
Status: newclosed
Note: See TracTickets for help on using tickets.