Opened 10 years ago

Closed 6 years ago

#10044 closed defect (patchwelcome)

Cannot set DataGrid.onRowClick directly, have to connect

Reported by: Shane O'Sullivan Owned by: Bryan Forbes
Priority: high Milestone: future
Component: DojoX Grid Version: 1.3.2
Keywords: Cc:
Blocked By: Blocking:


If a data grid is created and the onRowClick function is set directly, it does not decorate the event fully, and the rowIndex is not available.

To get around this it is neccessary to programmatically connect to the onRowClick function. This makes it impossible to use syntax such as:

<div dojoType="dojox.grid.DataGrid?" onRowClick="myFunction" ....

It would be better if all event decoration was performed in a private function, which then calls the stub onRowClick and onCellClick functions.

Change History (5)

comment:1 Changed 10 years ago by Nathan Toone

Milestone: 1.41.5
Owner: changed from Nathan Toone to Bryan Forbes

comment:2 Changed 9 years ago by bill

Milestone: 1.51.6

1.5 is released, moving remaining tickets to 1.6

comment:3 Changed 8 years ago by bill

Milestone: 1.6future

(sadly) punting seemingly abandoned ticket and meta tickets to future

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

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