dijit/Tree - TreeNode focus can break expando/plus click handler

[Mangala Sadhu Sangeet Singh Khalsa]

The fix for #17347 introduced a problem in IE (only tested in, and observed in IE8 and IE11).

If a node in the tree has a label that extends past the right edge of the tree (triggering horizontal scrolling), when you click the expando node, the a11yclick.press handler focuses the TreeNode? widget. In Firefox and Chrome, focusing does not cause scrolling. In IE, focusing scrolls the label (the focusNode) into view. This happens before the click handler for the expando node is called, and now the node under the mouse is no longer the expando node, so the value of event.target is incorrect and the node is not expanded.

See attached test case - problem occurs in Dojo 1.9.2 and 1.9.3.

[Colin Snover]

PR: ​https://github.com/dojo/dijit/pull/25

Bill could you please review? Thanks.

[bill]

Colin - Cool, looks good to me, thanks for the fix and the test.

[Colin Snover <[email protected]…>]

In 6b973fbf125199d03f9e957175ea7c61da4f18c3/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

[Colin Snover <[email protected]…>]

In bb512126abf359125c257db3d8b1be557186c0d8/dijit:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?