Closed Bug 471223 Opened 15 years ago Closed 15 years ago

Remove vestigal tag-anchor.gif

Categories

(Thunderbird :: Message Compose Window, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b2

People

(Reporter: philor, Assigned: philor)

Details

Attachments

(1 file)

Attached patch Fix v.1Splinter Review
Not quite up to my usual CVS archaeology standards, but:

At some point in the past, in the pre-1.0-era, before smileys were part of the theme, we decided to fork editor's packaging so we could package up just the smileys, instead of all the images in editor/ui/composer/content/icons/. Then after that, we realized that although we don't need things like tag-body.gif, which Composer shows when you enter the "Show icons for all HTML tags" mode that we don't have, <a name="foo"> shows tag-anchor.gif even in the normal view, so Insert - Named Anchor would have... probably shown a broken image icon, or maybe just nothing, so we added mail/components/compose/content/icons/tag-anchor.gif.

Then smileys moved to the theme, and (here's where my archaeology breaks down) whatever it was that was letting us find editor/content/ things in messenger.jar rather than comm.jar was removed, and we went back to packaging all of editor/ui/composer/content/icons/ (or maybe we never changed anywhere other than on the aviary branch), so now we're just jarring up something which nothing can access, since there's no .manifest saying to either look for content editor there, or to overlay any editor/content/ URI with anything from there.

(Doing some restriction of what composer bits and bobs we package wouldn't be a bad thing, but copying the files is the wrong approach: we should either ifdef editor/ui/jar.mn (particularly reasonable given the way we already do ifdef editor/ui/locales/jar.mn), or do what we do with mailnews/jar.mn, ifdeffing the whole thing and jarring what we want with absolute paths from a file in mail/ instead.)
Attachment #354525 - Flags: review?(mkmelin+mozilla)
Attachment #354525 - Flags: review?(mkmelin+mozilla) → review+
Comment on attachment 354525 [details] [diff] [review]
Fix v.1

Looks good to me, r=mkmelin
http://hg.mozilla.org/comm-central/rev/1c4718e13171
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b2
You need to log in before you can comment on or make changes to this bug.