Closed
Bug 471223
Opened 15 years ago
Closed 15 years ago
Remove vestigal tag-anchor.gif
Categories
(Thunderbird :: Message Compose Window, defect)
Thunderbird
Message Compose Window
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: philor, Assigned: philor)
Details
Attachments
(1 file)
1.89 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter 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)
Updated•15 years ago
|
Attachment #354525 -
Flags: review?(mkmelin+mozilla) → review+
Comment 1•15 years ago
|
||
Comment on attachment 354525 [details] [diff] [review] Fix v.1 Looks good to me, r=mkmelin
Assignee | ||
Comment 2•15 years ago
|
||
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.
Description
•