Closed
Bug 420003
Opened 17 years ago
Closed 17 years ago
Sort by name on tag container produces unexpected behavior
Categories
(Firefox :: Bookmarks & History, defect, P2)
Firefox
Bookmarks & History
Tracking
()
VERIFIED
FIXED
Firefox 3 beta5
People
(Reporter: zurtex, Assigned: ondrej)
References
Details
Attachments
(1 file, 1 obsolete file)
Latest trunk build as I post this:
1) Click "Smart Bookmarks" in bookmark toolbar
2) Go to recent tags
3) Right click on a tag container and sort by name
4) View horrific result
Restart restores titles...
Flags: blocking-firefox3?
Reporter | ||
Comment 1•17 years ago
|
||
Just test on Linux, exactly the same problem, so I assume it's fairly safe to say it's cross platform.
OS: Windows Server 2003 → All
Priority: -- → P2
Hardware: PC → All
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Updated•17 years ago
|
Assignee: nobody → ondrej
Target Milestone: --- → Firefox 3
Assignee | ||
Comment 2•17 years ago
|
||
Order of query based queries should not be modifiable. The solution for this bugs is to disable this command.
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•17 years ago
|
||
(In reply to comment #2)
> Order of query based queries should not be modifiable. The solution for this
> bugs is to disable this command.
>
We have 3 places where we can see tag containers.
A) Bookmark Toolbar -> Smart Bookmarks -> Recent Tags
B) Places Library -> All Bookmarks (and the same as above)
C) Places Library -> Tags
The set of commands seems to be the same for A and C, with the same result when sorting (fortunately titles are not really deleted, just nulled in memory). In case B they are not modifiable.
But there are more problems, it seems strange to me, that it is allowed to create a new folder or new bookmark inside of tag container. When I right click on the right side I can even create a separator. Creating a new bookmark this way automatically shows as if it was tagged this way (what seems correct).
My suggestion for this bug:
- disallow adding new content to the tag containers everywhere,
- fix sort by name to work properly (probably only annotation versus bookmark title problem).
Assignee | ||
Comment 4•17 years ago
|
||
The problem of empty titles is caused by wrong query in mDBBookmarkToUrlResult (nsNavHistory.cpp). It does not perform the same COALESCE on places title as mDBGetChildren does (this may get fixed by bug 419731, but I decouple those two for a moment).
Another problem is sorting. Sort by name actually works - it changes the position of included links, however, the bookmark query that displays them in "Smart Bookmarks" sorts by SORT_BY_DATE_DESCENDING. Sorting will be removed to use the original one. Unfortunately this will not solve the sorting problem due to bug 359332.
No longer depends on: 419731
Summary: Sort by name fails and removes titles for tag containers → Sort by name on tag container removes titles of tagged links and does not show correct order
Assignee | ||
Comment 5•17 years ago
|
||
This does not solve the bug completely, but it solves the scarifying problem of disappearing titles. The fact that sort order is not visible should be solved with bug 359332 (which I will nominate to block Fx3).
Attachment #307260 -
Flags: review?(dietrich)
Updated•17 years ago
|
Attachment #307260 -
Attachment mime type: application/octet-stream → text/plain
Comment 6•17 years ago
|
||
Comment on attachment 307260 [details]
Fix disappearing titles
r=me. please note on bug 359332 that the sorting options removed here need to be re-added when it gets fixed.
Attachment #307260 -
Flags: review?(dietrich) → review+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Comment 7•17 years ago
|
||
Checking in toolkit/components/places/src/nsNavHistory.cpp;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistory.cpp,v <-- nsNavHistory.cpp
new revision: 1.267; previous revision: 1.266
done
Comment 8•17 years ago
|
||
verified with:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5pre)
Gecko/2008032106 Minefield/3.0b5pre
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 9•17 years ago
|
||
The current behavior for this is even worse, currently all bookmarks are removed!
But given that tag behavior is changing, I'm not sure what can be done about this at the moment, see bug 419731.
Status: VERIFIED → REOPENED
Depends on: 419731
Resolution: FIXED → ---
Summary: Sort by name on tag container removes titles of tagged links and does not show correct order → Sort by name on tag container produces unexpected behavior
Reporter | ||
Comment 10•17 years ago
|
||
Sorry, I'd like to update the behavior here.
All the bookmarks aren't removed when sorting by name, just 1 of them. However when I tested this there was only 1 bookmark in that selected tag, so it got removed, tested on:
Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9b5pre) Gecko/2008032106 Minefield/3.0b5pre ID:2008032106
Reporter | ||
Updated•17 years ago
|
Attachment #306182 -
Attachment is obsolete: true
Comment 11•17 years ago
|
||
I didn't see that on Win XP when I verified this bug.
Please file a new bug.
Status: REOPENED → RESOLVED
Closed: 17 years ago → 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Status: RESOLVED → VERIFIED
Updated•17 years ago
|
Target Milestone: Firefox 3 → Firefox 3 beta5
Reporter | ||
Comment 12•17 years ago
|
||
(In reply to comment #11)
> I didn't see that on Win XP when I verified this bug.
>
> Please file a new bug.
>
Well I got someone else to confirm it on XP, but will file a new bug anyway.
Reporter | ||
Comment 13•17 years ago
|
||
Made bug 424648
Please try confirming there, I can confirm 100% of the time on different computers and have got other people to confirm on different OS'.
Comment 14•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
•