Opened 9 years ago
Closed 8 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)
Change History (4)
Changed 9 years ago by
Attachment: | test3.html added |
---|
Changed 9 years ago by
Attachment: | json_table.php added |
---|
comment:1 Changed 8 years ago by
comment:2 Changed 8 years ago by
Resolution: | → patchwelcome |
---|---|
Status: | new → closed |
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.