Opened 4 years ago

Closed 4 years ago

#18657 closed defect (patchwelcome)

onToggle in dojox.grid._LazyExpando stop click event

Reported by: ShaoFeng Owned by:
Priority: undecided Milestone: tbd
Component: DojoX Grid Version: 1.10.4
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by bill)

dojox/grid/LazyTreeGrid.js:

	onToggle: function(evt){
		// summary:
		//		The onclick handler of expando, expand/collapse a tree node if has children.
		if(this.grid._treeCache.items[this.rowIdx]){
			this.grid.focus.setFocusIndex(this.rowIdx, this.cellIdx);
			this.setOpen(!this.grid._treeCache.items[this.rowIdx].opened);
			try{
				event.stop(evt);           <=====================
			}catch(e){}
		}
	},

The event.stop(evt) stop click event from propagating.

So user click on the expando can not select the row.

http://demos.dojotoolkit.org/dojox/grid/tests/test_treegrid_lazyloading.html has also this problem.

Change History (1)

comment:1 Changed 4 years ago by bill

Component: GeneralDojoX Grid
Description: modified (diff)
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.