Invalid iconClassPrefix in TablePlugins.js and UploadImage.js

[oliver_]

The Dojox Plugins TablePlugins? and UploadImage? have a wrong iconClassPrefix set.

Instead of iconClassPrefix:"editorIcon" it should be iconClassPrefix:"dijitEditorIcon" in line 214 of dojox/editor/plugins/TablePlugins.js and line 14 of dojox/editor/plugins/UploadImage.js, respectively.

This bug causes the icons for the buttons of these plugins (in the Dijit Editor) to be absent in Dojo Version 1.3.0 due to the lack of a editorIcon CSS class.

[Adam Peller]

fixed?

[dante]

yep, in [19067] -- postCommit hooks must be b0rked. sorry about that.

[oliver_]

Actually I'm not sure whether this change is a good idea. Having added more buttons from the table plugin, I noticed they were absent too, which prompted me to dig deeper into the code which made me realize that there is only a class for dijitEditorIconInsertTable, but not for dijitEditorIconModifyTable and so on. I found that for the plugins, there is a separate CSS file existing in dojox/editor/plugins/resources/editorPlugins.css defining the needed classes editorIconInsertTable, editorIconModifyTable, ...

Sorry about that, I'm quite new to using Dojo and am not quite aware of the underlying structures yet. Since the editor is skinnable, I would have supposed it uses the skin CSS at dijit/themes instead of a separate CSS, where the icons stand out quite a bit from the rest of the design. So I suppose that's no problem at all, and reverting changes and loading the separate CSS should do. I'm sorry for the trouble!

[dante]

no trouble, this is a documentation fail. The css is linked in the two tests, but they don't follow ANY of our conventions (mixed tab/whitespace, tests are prefixed with test_, inline comments in the test, etc).

@mike, can you please clean this up. svn blame says you own him :)

[dylan]

Bug was fixed in 1.4. Ticket was left open to clean-up test code, which hasn't happened. If someone wants to clean that up, please open another ticket.