717240 - Remove incomplete add/edit image map code from editor

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

There's a bunch of code in editor/ui for adding/editing image maps.

I came across this whilst working on bug 700808 - the strings for the buttons are in non-suite specific ifdef, but the partial image map UI strings are in a suite specific ifdef.

I think we should just remove this code for now. Its obvious it doesn't work, and if we want to make it work later we can always pull it out of history. This also gives about 56 strings less for localisers to translate.

I have left the remove image map button in - afaict that would work.

Comment 1

[Ian Neal]

There is also a couple of css files in each of the suite themes, also referenced in an inspector test, that could probably be removed but that could be in a follow up:
EdImageMap.css
EdImageMapPage.css

Comment 2

[[e-mail]]

Comment on attachment 587700 [details] [diff] [review]
The fix

Yeah, the code is ancient and nowhere near working, so...

Comment 3

[Mark Banner (:standard8)]

Attachment: The fix v2

Removes the css files as well. As far as I can tell Inspector doesn't actually use allskin.xul, so I'm going to file a bug about changing it or removing it.

The images referenced in the css files don't seem to exist either.

Comment 4

[Ian Neal]

Comment on attachment 588127 [details] [diff] [review]
The fix v2

r=me and thanks

Comment 5

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/81c392144beb