event.stopEvent method causes error on IE when using frames

[guest]

When the event originates from a different window then window.event is null and therefor in

if(window.event){
   evt.cancelBubble = true;
   evt.returnValue = false;
}else{
   evt.preventDefault();
   evt.stopPropagation();
}

the else block is executed which yields a "Object doesn't support this property or method" error.

I think the "if" clause should be changed to

   if(typeof(window.event) == "undefined"){
   ... 

[bill]

In 0.9 this code has been changed to:

	dojo.stopEvent = function(/*Event*/evt){
		// summary:
		//		prevents propagation and clobbers the default action of the
		//		passed event
		// evt: Optional for IE. The native event object.
		evt.preventDefault();
		evt.stopPropagation();
	}

Looks fixed. The comment is obviously wrong though so I'll fix that.

[bill]

(In [9823]) Fix comments (evt is obviously not optional). Refs #2556.

[sjmiles]

Farther down in the file we have:

365		// override stopEvent for IE
366		dojo.stopEvent = function(evt){
367		evt = evt || window.event;

So the documentation was actually correct, although (afaik) it's an open question how the documentation engine handles such overrides.

[sjmiles]

(In [9828]) Comments hopefully clarified, refs #2556.

[Adam Peller]

(In [9829]) Minor style, spelling changes. Refs #2556