MenuBar: tabIndex problems after focusing programatically

[hyang]

1: get test_Menu.html source code under folder dijit\tests

2: add method below to move focus to menubar with CTRL+M

       function addListenerToDoc(event){
		if(event.ctrlKey && event.charCode == 109){//Ctrl+M
			var menubar = dijit.byId('menubar');
	    	var menu = dijit.byId('file');
	    	if(menubar && menu){
	    		menubar._markActive();
	    		if(menubar.focusedChild) return;
	    		menubar.focusChild(menu);
	    		if(menu.popup)
	    			menubar._openPopup();
	    	}
		}
	}

3: call the method in addOnLoad

4: browser the page, and press CTRL+M to move focus to menubar

5: press ESC to close the popup, the focus on the file menu

6: press Shift+ tab

Defect: the focus still on the file menu

[hyang]

source code

[bill]

Hmm, yes, Menu (and anything derived from _KeyNavContainer) is not designed to get focus programatically.

Originally the container has a tabIndex, and when it receives focus via the tab key, it immediately shifts focus to it's first child, and sets it's own tabIndex to -1 (or removes it's tab index). This is what _onContainerFocus() does.

But in this case _onContainerFocus() isn't called, so the shift-tab moves from the File MenuItem to the container itself.

[bill]

In [29911]:

Make sure container's tabindex is removed whenever one of the container's children gets focus, regardless of how it got focus. Fixes #16300 !strict.

[bill]

In [29924]:

just fixing bug number in comment, refs #16300.