Closed
Bug 408660
Opened 17 years ago
Closed 16 years ago
Should be able to set icons for special folders in the organizer
Categories
(Firefox :: Bookmarks & History, defect, P1)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
FIXED
Firefox 3
People
(Reporter: asaf, Assigned: asaf)
References
Details
Attachments
(1 file, 1 obsolete file)
2.35 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
See organizer mockup. We should allow themes to set icons for special folders in the organizer (in particular, all the left pane queries).
Flags: blocking-firefox3?
Updated•17 years ago
|
Flags: wanted-firefox3+
Flags: blocking-firefox3?
Flags: blocking-firefox3-
Summary: Should be to set icons for special folders in the organizer → Should be able to set icons for special folders in the organizer
Updated•16 years ago
|
Target Milestone: Firefox 3 beta3 → ---
Comment 2•16 years ago
|
||
Mano, can we borrow Ryan's patch from bug 424951 here?
Flags: blocking-firefox3- → blocking-firefox3+
Assignee | ||
Comment 3•16 years ago
|
||
I already have a better fix in hand. patch coming.
Assignee | ||
Comment 4•16 years ago
|
||
Attachment #314200 -
Flags: review?(dietrich)
Assignee | ||
Comment 5•16 years ago
|
||
Attachment #314200 -
Attachment is obsolete: true
Attachment #314201 -
Flags: review?(dietrich)
Attachment #314200 -
Flags: review?(dietrich)
Assignee | ||
Updated•16 years ago
|
Priority: -- → P1
Target Milestone: --- → Firefox 3
Comment 6•16 years ago
|
||
Comment on attachment 314201 [details] [diff] [review] patch >+ if (itemId != -1) { >+ var oqAnno; >+ try { >+ oqAnno = PlacesUtils.annotations.getItemAnnotation(itemId, ORGANIZER_QUERY_ANNO); >+ properties.push(this._getAtomFor("OrganizerQuery_" + oqAnno)); >+ Components.utils.reportError(oqAnno); what's up with this last line? also, please document the values somewhere.
Assignee | ||
Comment 7•16 years ago
|
||
will remove it on checkin.
Comment 8•16 years ago
|
||
Comment on attachment 314201 [details] [diff] [review] patch r=me w/ that line removed
Attachment #314201 -
Flags: review?(dietrich) → review+
Updated•16 years ago
|
Version: unspecified → Trunk
Assignee | ||
Comment 9•16 years ago
|
||
mozilla/browser/components/places/content/treeView.js 1.61
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Comment 10•16 years ago
|
||
verified with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9pre) Gecko/2008041804 Minefield/3.0pre
Status: RESOLVED → VERIFIED
Comment 11•15 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
•