Closed
Bug 382095
Opened 17 years ago
Closed 16 years ago
Remove obsolete places theme files
Categories
(Firefox :: Bookmarks & History, defect)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
Firefox 3
People
(Reporter: dietrich, Assigned: philor)
References
Details
Attachments
(1 file, 1 obsolete file)
5.25 KB,
patch
|
asaf
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
both directories contain some obsolete files as well as some files in use. all files in use should be copied into the places dir, and all obsolete files removed from the places dir. old /bookmarks files should remain for in place for non-places bookmarks builds.
Reporter | ||
Updated•17 years ago
|
Target Milestone: Firefox 3 alpha6 → Firefox 3 beta1
Reporter | ||
Updated•17 years ago
|
Target Milestone: Firefox 3 M7 → Firefox 3 M8
Reporter | ||
Updated•17 years ago
|
Target Milestone: Firefox 3 M8 → Firefox 3 M9
Reporter | ||
Updated•16 years ago
|
Target Milestone: Firefox 3 M9 → Firefox 3 M10
Reporter | ||
Updated•16 years ago
|
Target Milestone: Firefox 3 M10 → Firefox 3 M11
Reporter | ||
Updated•16 years ago
|
Target Milestone: Firefox 3 beta3 → ---
Assignee | ||
Comment 1•16 years ago
|
||
Despite having had the email from this starred in my bugspam folder for almost a year, I wound up doing the last bit of consolidate and remove /bookmarks/ in bug 424613, so now I'm desperately searching for something obsolete in /places/ besides the venerable placesIcon.png, so I don't have to *just* kill the goldfish blowing a bubble here.
Depends on: 424613
Summary: consolidate theme files in the bookmarks and places dirs → Remove obsolete places theme files
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → philringnalda
Assignee | ||
Comment 2•16 years ago
|
||
Plenty o' review, so I don't kill something you're planning on bringing back next week, but these are the things that don't show any sign of being used.
Attachment #316537 -
Flags: review?(mano)
Attachment #316537 -
Flags: review?(dietrich)
Comment 3•16 years ago
|
||
notice that new icons has been landed to be used soon (bug 429282) but are still unused. for example calendar.png will be soon used for day containers, so you should not remove it
Assignee | ||
Comment 4•16 years ago
|
||
Comment on attachment 316537 [details] [diff] [review] Fix v.1 Sigh. And libraryNavigation.png was pre-landed for bug 428924. Among the many things we've tried that don't entirely work out, I'd list this "land icons by the gobful" technique. (And "use a different filename for the very same thing on different platforms," too.)
Attachment #316537 -
Attachment is obsolete: true
Attachment #316537 -
Flags: review?(mano)
Attachment #316537 -
Flags: review?(dietrich)
Assignee | ||
Comment 5•16 years ago
|
||
Quite a bit less - since calendar.png and libraryNavigation.png have uses coming, collapse.png and expand.png might (I'll brace faaborg about them separately), and I really hope pinstripe's starPage.png is going to, we're down to: pinstripe: * infoPaneGrippy.png went away with bug 403140 * contentSplitter-bg.gif would have, except it seems to have landed unused * folderDropHoverArrow.png is the white dropmarker, from back when hovered bookmark folders had such a dark background it needed to be white * menubutton-mid.png must have been from some plan that involved stretching, which isn't currently being used * the tabs in the plus and minus lines give me a rash * poor old placesIcon.png needs to be put out to pasture winstripe * poor old placesIcon.png lost his identity, getting turned into a star, so he really needs to be put out to pasture
Attachment #316648 -
Flags: review?(mano)
Comment 6•16 years ago
|
||
Comment on attachment 316648 [details] [diff] [review] Fix v.2 r=mano
Attachment #316648 -
Flags: review?(mano) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #316648 -
Flags: approval1.9?
Comment 7•16 years ago
|
||
Comment on attachment 316648 [details] [diff] [review] Fix v.2 a1.9=beltzner
Attachment #316648 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Comment 8•16 years ago
|
||
browser/themes/pinstripe/browser/jar.mn 1.84 browser/themes/pinstripe/browser/places/contentSplitter-bg.gif delete browser/themes/pinstripe/browser/places/folderDropHoverArrow.png delete browser/themes/pinstripe/browser/places/infoPaneGrippy.png delete browser/themes/pinstripe/browser/places/menubutton-mid.png delete browser/themes/pinstripe/browser/places/placesIcon.png delete browser/themes/winstripe/browser/places/placesIcon.png delete
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
Comment 9•14 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•