Opened 6 years ago

Closed 5 years ago

#16895 closed defect (patchwelcome)

'rowHeightChanged' of _Grid.scroller is undefined

Reported by: chrisacky Owned by: Evan
Priority: undecided Milestone: tbd
Component: DojoX Grid Version: 1.8.3
Keywords: Cc:
Blocked By: Blocking:

Description

Hey, With dojox.EnhancedGrid? (might also be in DataGrid?) in file dojox/grid/_Grid.js on line 964, if you reuse the same grid multiple times, and clear out the items, then use the grid again when it doesn't have a scroller, it appears as though the scroller becomes undefined.

Checking that scroller exists would fix it...

this.scroller.rowHeightChanged(inRowIndex);

Someone with more knowledge than me on the internals could probably advise, but for me, scroller is undefined after reusing the Grid multiple times. (ie, Clearing the store and inserting new items so the scroll doesn't show).

typeof this.scroller !== "undefined" would fix this.

Change History (1)

comment:1 Changed 5 years ago by bill

Resolution: patchwelcome
Status: newclosed

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.

Note: See TracTickets for help on using tickets.