disable new bookmark / new live bookmark when excludeItems is set spun off from bug #383681, per mano note, File | New Live Bookmark should be disabled as well.
11 years ago
Depends on: 383681
Created attachment 271914 [details] [diff] [review] patch Depends on nodeExcludesItems() in utils.js, in the patch for bug 387035
Attachment #271914 - Flags: review?(sspitzer)
hrm, I don't think this is right for live bookmarks actually... these are visible in exculdeItems queries.
Attachment #271914 - Flags: review?(sspitzer) → review-
Created attachment 271918 [details] [diff] [review] patch (In reply to comment #2) > hrm, I don't think this is right for live bookmarks actually... these are > visible in exculdeItems queries. Makes sense, you're right.
Comment on attachment 271918 [details] [diff] [review] patch Oh well, just make this this._view.getResult().root.queryOptions.excludeItems And please remove the asQuery wrapping in the separator case, GetRoot returns the container as a query a object.
Created attachment 271925 [details] [diff] [review] revision
Comment on attachment 271925 [details] [diff] [review] revision r=mano
Attachment #271925 - Flags: review? → review+
Are we sure we want to disable these commands when excludeItems is set? Unlike separators (see bug #383681), no matter what the sort is on the right hand pane, adding a new livemark / new bookmark, still makes sense right? It seems that disabling the commands just because a folder in the left hand pane has focus may not be correct. (How would an end user know that they right hand pane needs focus for those commands to work?) Additionally, what would that mean for paste when the left hand pane has focus? asaf / cyen, what do you think?
Comment on attachment 271925 [details] [diff] [review] revision Also, what did fx 2 do?
> Also, what did fx 2 do? For fx 2, new bookmark, new folder, and new live bookmark are all enabled. I think we should WONTFIX this bug. cyen, what do you think?
(In reply to comment #9) > > Also, what did fx 2 do? > > For fx 2, new bookmark, new folder, and new live bookmark are all enabled. > > I think we should WONTFIX this bug. > > cyen, what do you think? > I think... with the patch for bug 387035, inserting with anything in the left panel selected will insert the item correctly into the bottom of the folder's contents, in the right pane - and the problematic "my-new-item-isn't-showing-up-where-it's-supposed-to" behavior in bugs like bug 383681 will no longer be a problem. Actually - I sort of want to enable separators if excludeItems = true, too, for the sake of consistency.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WONTFIX
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.