Editor table plugin DOA in IE7

[haysmark]

​http://archive.dojotoolkit.org/nightly/checkout/dojox/editor/tests/editorTablePlugs.html

Load the page in IE7. Before the editor renders you get the error: "this.editorDomNode has no properties." The parser continues to load the Editor widget but without the table controls at the top.

The debugger points to these lines in dojox/editor/plugins/TablePlugins.js:

// IE ONLY
this.editorDomNode.ondragstart = dojo.hitch(this, "onDragStart");
this.editorDomNode.ondragend = dojo.hitch(this, "onDragEnd");

It's odd that "IE ONLY" code would fail in IE.

[Mike Wilcox]

I really can't do anything with the table plugins until I hear something about the new editor.

[Adam Peller]

Mike, I think the assumption should be that there is no new editor until you hear otherwise. How can we make what is in trunk functional?

[Mike Wilcox]

Part of the problem between the two is how they handle focus, the first being iframe based, the second being div based. I think more of the issue is when I first made it, FF only supported iframe based editors. But then it was IE that threw the errors; go figure.

The more pertinent problem is with work I currently have no bandwidth, and what little I have I tend to lean toward the FileUploader?. I will certainly tackle this when I get time, but I'd rather make it future leaning toward the new editor than toward the old.

[Adam Peller]

not so fast (please note priority setting of 'high') there is broken code in the trunk. let's figure out why it's broken before we defer, or whether it should be removed/doc'd.

[Mike Wilcox]

Fixed with r16351

Fixes #1778 - did a better targeting of the iframe or editNode - even though they are the same thing, the browsers are funky about it. Everything seems to work pretty well in IE and FF, with the exception of cell color in IE completely fails. Over all good results though. Note to self: this plugin works with iframe-based editors only.