Move dojox.form._HasDropDown to dijit

[Nathan Toone]

The following things need to happen:

1 - Rename and flag as protected the following members:

dropDownNode -> _buttonNode
aroundNode -> _aroundNode
popupStateNode -> _popupStateNode

2 - Remove _onMenuMouseup from _HasDropDown
3 - Move the dijit.Menu-specific handling code from dojox.form.DropDownSelect? into _HasDropDown
4 - Move _HasDropDown to dijit._HasDropDown, and create a stub (deprecated) in dojox.form (in case anyone is using it)
5 - Upgrade the following dijit components to use _HasDropDown instead of their current code:

dijit.form.DropDownButton
dijit.form.ComboButton (extends from DropDownButton)
dijit.form.ComboBox
dijit.form.FilteringSelect (extends from ComboBox)
dijit.form._DateTimeTextBox
dijit.form.DateTextBox (extends from _DateTimeTextBox)
dijit.form.TimeTextBox (extends from _DateTimeTextBox)

6 - Cross fingers and wait for breakage reports to come in :)

[bill]

(In [17647]) Comment format/content update as per explanations from toonetown. Plus a few TODO/TODOC's.

Refs #9356 !strict

[Nathan Toone]

Patch which does the first 4 of these tasks (and also move dojox.form.DropDownSelect? to dijit.form.DropDownSelect?)

[Nathan Toone]

(In [19071]) Refs #9356 - initial 4 steps towards completing the migration of DropDownSelect?/_HasDropDown to dijit !strict

[Nathan Toone]

(In [19305]) Refs #9356 - do step 5 for DropDownButton? and ComboButton?

[liucougar]

(In [19511]) refs #9356: fixed broken ComboButton? popupStateNode is renamed to _popupStateNode in ComboButton? in r19305

[bill]

(In [19519]) just saving a few bytes, refs #9356 !strict.

[bill]

(In [19523]) Correctly account for page scroll when testing if the drop down menu has appeared over the drop down button, and the user has moused-up w/out moving off of the button. (BTW this can be tested on test_Button.html by making the browser window very short and then scrolling down to the first row of buttons)

Also switching from dojo.coords() to dojo.position().

Refs #9356, #9648 !strict

[bill]

(In [20034]) Add DropDownSelect? test to suite, refs #9356.

[Nathan Toone]

Closing this as fixed - a separate bug will be opened for the migration of combobox, filtering select, and date/time text boxes. I don't want to destabilize those so close to 1.4...

[bill]

New ticket is #9945.

[Becky Gibson]

(In [20238]) refs #9356 updated with more appropriate aria roles and states. Updated the test file to include label elements for accessibility.

[bill]

(In [20283]) Regression from [20238], need to define jsId=testDisabled for the DOH test to run. Plus fixed some typos. Refs #9356.

[bill]

(In [20822]) Updating comments, refs #9356 !strict.

[bill]

(In [20944]) Add dijit.form.Select to themeTester, plus adding some menu items for memory leak testing and theme switching. Refs #9356.

[bill]

(In [20945]) On second thought, put dijit.form.Select in a new tab, since it can't qualify as a "text input widget". And put ComboBox in the new tab too, so Select isn't lonely.

Also splitting Editor into a separate tab.

Refs #9356.

[bill]

(In [21018]) Missing dojo.require() calls. These are loaded anyway via dijit-all.js but it's proper to have the dojo.require() calls. Refs #9356

[bill]

(In [21203]) Don't orphan the drop down widget from the DOM tree; it causes TooltipDialog to prematurely load it's href (before the widget is visible), and also possibly other obscure problems, like if the app has dojo.query() code that runs on page load, and expects to find nodes in the drop down widget.

Fixes #10652, refs #9356 (the code which prevents the orphaning was removed in [19305] but I'm not sure why), !strict.

[bill]

(In [22603]) Undo() is inherited from _FormValueWidget, no need to repeat the definition in _FormSelectWidget. Refs #9356 !strict.

[bill]

In [27461]:

Add back dijitDropDownButtonOpened class that used to be written before the conversion to use _HasDropDown, fixes #14524, refs #9356 !strict.