Adding the original event to the onClick call in Tree widget

[genro]

I am suggesting here to add the parameter 'event' to the onClick call in the Tree widget as in many circumstances the user interface could use the event modifiers to perform a more specific action. So for example a Shift click on a tree item could act differently than a pure click or an alt + click. It can be easily implemented to dijit Tree with the following changes :

line 882:

• this.onClick(nodeWidget.item, nodeWidget);

+ this.onClick(nodeWidget.item, nodeWidget, e);

line 929

• onClick: function(/* dojo.data */ item, /*TreeNode?*/ node)

+ onClick: function(/* dojo.data */ item, /*TreeNode?*/ node,/*Event*/ e)

The same type of behavior could be implemented also for DblClick? and in a more general way when Dijit wraps an event it should pass the original event to the wrapper to allow a better tuning of the interface. So maybe it could be useful also for other widgets.

Thank you

Giovanni Porcari

[bill]

I checked the other widgets and most/all of them are passing the event already, but I'll update Tree to do so too.

[bill]

Fixed in [17523].

[liucougar]

(In [23483]) refs #9289: properly pass the keypress event to Tree._onEnterKey !strict