Tree: focus not updated correctly in various cases

[bill]

ISTM focus on the tree isn't updated correctly in the following cases (or at least some of the following cases):

• focused node is deleted (by, for example, another user... and Tree receives delete notification)
• focused node's ancestor is deleted
• focused node is dragged and dropped into collapsed node, thus becoming hidden

Does focus go to "the right place" in these cases (whatever that means)? Or does it just disappear (or is that the "right thing")? Or does it move to another widget entirely, outside of tree?

Closing the ancestor of focused node seems to do the right thing:

if(this.lastFocused){
	// are we collapsing a descendant with focus?
	if(dojo.isDescendant(this.lastFocused.domNode, node.domNode)){
		this.focusNode(node);
	}else{
		// clicking the expando node might have erased focus from
		// the current item; restore it
		this.focusNode(this.lastFocused);
	}
}

[bill]

Mike asked me for a fix to (1) and (2), I'm working on it now. Actually he just asked to avoid the exception when the user clicks on another TreeNode after the selected one was deleted out from under them. But I want to fix the tabbing problem too.

Oh, turns out the third bullet point is impossible:

focused node is dragged and dropped into collapsed node, thus becoming hidden

because when you drop onto a collapsed node it automatically expands, and also because dropping sets the focus to the drop target.

[bill]

(In [20818]) Prevent exception when a TreeNode is programatically deleted and then the user switches focus to another node (as seen in test case for #10018), refs #6236 !strict.

[bill]

Mostly fixed by [30029].

[dylan]

If it's mostly fixed, what remains to be done? Or should be close this out?

[bill]

See #16323. Doug complained about the new behavior in [30029] so I reverted it.

Also, there's a problem when a node is destroyed while it actually has focus. As opposed to destroying the node marked to get focus when the Tree itself gets focus.

[Bill Keese <bill@…>]

In 8cc12c35ee8a6d512c65339771396574224f1a40/dijit:

Error: Processor CommitTicketReference failed

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

[Bill Keese <bill@…>]

In fb8e1093/dijit:

Error: Processor CommitTicketReference failed

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

[Bill Keese <bill@…>]

In b87bfbe/dijit:

Error: Processor CommitTicketReference failed

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

[Bill Keese <bill@…>]

In f35a55a/dijit:

Error: Processor CommitTicketReference failed

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