Multiple Editor2 instances do not work properly in Firefox

[liucougar]

in ​http://archive.dojotoolkit.org/nightly/tests/widget/test_Editor2.html, try select some text in the first instance of the Editor2, click "bold" button: no effect!

While select some text in the second instance, it works fine

After investigation, I think the issue is caused by the fact that this.EditNode? is not initialized when these codes are executed:

dojo.event.topic.registerPublisher("Editor2.clobberFocus", this.editNode, "onfocus");
dojo.event.topic.subscribe("Editor2.clobberFocus", this, "setBlur");
dojo.event.connect(this.editNode, "onfocus", this, "setFocus");

However, I tried several approaches to workaround this with no success. Any clues?

[liucougar]

Finally, I think I solved it:

Just moving the editorOnLoad() to a later stage when this.EditNode? is set correctly won't solve this: in firefox, browser specific features are used to handle onFocus event in RichText?:

560        dojo.event.browser.addListener(this.document, "focus", dojo.lang.hitch(this, "onFocus"));

So dojo.event.connect won't work anymore. Attached is a patch to rectify this: an global variable, dojo.widget.html._CurrentEditor2Instance, is introduced to save the current focused editor2 instance (this is required for Firefox as it generates a onfocus event for every click in the editing area) Editor2 overload onFocus() in RichText? now to handle setFocus correctly some Useless codes are removed as well

I also tested this patch against IE without noticable issues (I have to remove 'useActiveX="true"' in the first instance to load the page)

[liucougar]

(sorry, the format in the previous comment seems messed up, repost it here)

So dojo.event.connect won't work anymore. Attached is a patch to rectify this:

an global variable, dojo.widget.html._CurrentEditor2Instance, is introduced to save the current focused editor2 instance (this is required for Firefox as it generates a onfocus event for every click in the editing area)

Editor2 overload onFocus() in RichText? widget now to handle setFocus correctly some Useless codes are removed as well

I also tested this patch against IE without noticable issues (I have to remove 'useActiveX="true"' in the first instance to load the page)

[liucougar]

fixed in svn r4498

[(none)]

Milestone 0.4 deleted