Dialog: consecutive Dialog hide and show issue

[guest]

Creating 2 dialog boxes and consecutively showing and hiding of each leads to a show() that displays nothing with greyed back ground.

example. Click link over and over till issue.

			dojo.require("dojo.widget.*");
			dojo.addOnLoad(onload);
			function onload()
			{
			var parentnode = document.createElement("div");
			var srcNode = document.createElement("div");
			parentnode.appendChild(srcNode);
			dlg1=dojo.widget.createWidget("Dialog",{toggle:"fade"},srcNode);
			dlg1.setContent("<a href='#' onclick='dlg2test()'> dlg1 </a>");

			var srcNode2 = document.createElement("div");
			parentnode.appendChild(srcNode2);

			dlg2=dojo.widget.createWidget("Dialog",{toggle:"fade"},srcNode2);
			dlg2.setContent("<a href='#' onclick='dlg1test()'> dlg2 </a>");
				
			dlg1.show();
			
			}
			function dlg1test()
			{
				dlg2.hide();
				dlg1.show();
			}
			function dlg2test()
			{
				dlg1.hide();	
				dlg2.show();
			}

[alex]

Milestone 2.0 deleted

[bill]

Yes, I assume the problem is that you are trying to show the second dialog before the first one has finished disappearing (or vice-versa).

[bill]

Another test case and problem w/underlay in #10257.

[bill]

underlay disappearing problem from #10257

[bill]

(In [22662]) Refactoring Dialog to handle when multiple dialogs are displayed and a non-top-level dialog is hidden or destroyed. The salient change is that rather than a Dialog saving the previous focus, there's a "level manager" that remembers each active layer (starting with the base page itself) and the focus in that layer.

Also fixes race conditions on concurrent dialog show/hide.

Previously Dialog._onKey was connected to the document body. I couldn't find a case when that was necessary (as opposed to connecting to Dialog.domNode), so I changed it to attach to Dialog.domNode, so Dialog wouldn't have to check "am I the top dialog on the stack?" in that method.

Fixes #2238, #9944, #10705 !strict.

[ben hockey]

not sure which one of the 3 tickets closed by r22662 i should comment on so just picking one...

i think this line: pd.dialog._getFocusItems(pd.domNode);

might need to be: pd.dialog._getFocusItems(pd.dialog.domNode);

if by just looking at it you're not sure then let me know and i'll work on a test case.

[bill]

Thanks, you are right, I'll check that in w/a test case. (As for all the tickets, I should have consolidated them into one.)

[bill]

(In [22774]) Fix typo in code for corner case where two dialogs display near simultaneously and the lower dialog never gets focus. Fixes #2238 !strict.

[luis]

Hi, I am having the same problem that #10257; I would like to know if there is fixed with #2238. I tried it but still not working. Thanks

[bill]

#10527 is fixed; you can try the attached test case, updating it to run against the latest dojo code, and it works. If you are having a problem with Dialog you need to build a reproducible test case, and open a new ticket, attaching the test case, like #10527 did.

[bill]

(In [23458]) Test case doesn't require parser. Also fixing tabbing and upgrading to HTML5 DOCTYPE.

Refs #2238.