Grid -- text box -- single click does not clear selection

[guest]

Picky, but users who have to use this all day long find this very annoying:

When text in a grid edit box is selected, single clicking in the text does *not* deselect the text and place the cursor.

To reporoduce: go to

dojo/dojox/grid/tests/test_edit.html

in the top row, double click "But are not followed by two hexadecimal" and then single click the "not" as if to deleted it (or change a spelling mistake)

the single click does nothing and the text remains selected. You must use an arrow key to deselect the text.

This is especially annoying since the text automatically selects when double clicking and opening up the grid edit -- removing the "select" below seems appease users, but this still seems like a bug:

dojox.grid.focusSelectNode = function(inNode) {

try{

dojox.grid.fire(inNode, "focus"); dojox.grid.fire(inNode, "select"); -DL Selecting all of the text annoys users

}catch(e){ IE sux bad }

};

[benschell]

The primary issue here is that the click-handling is putting the focus on the cell (rather than let it fall through to the native editor nodes). Patch attached which, if the current event's target is the cell that is currently being edited, doesn't squash the focus.

[Jared Jurkiewicz]

Minor stylistic update.

[Jared Jurkiewicz]

(In [12516]) Minor patch to grid editing selection events from Ben Schell. fixes #5571

[Jared Jurkiewicz]

Tested on:

Firefox 2.0.0.12

IE 6

Safari B3