Closed Bug 1432604 Opened 6 years ago Closed 4 years ago

Remember the last used folder when bookmarking through Firefox's UI

Categories

(Firefox :: Bookmarks & History, enhancement, P5)

58 Branch
enhancement

Tracking

()

RESOLVED FIXED
84 Branch
Tracking Status
firefox84 --- disabled
firefox85 --- fixed

People

(Reporter: tbnguyen25, Assigned: Gijs)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:58.0) Gecko/20100101 Firefox/58.0
Build ID: 20180118215408

Steps to reproduce:

Press CTRL+D or clicking the bookmark star in the search field.


Actual results:

The folder to save the bookmark always default to "Other bookmarks" instead of the previously saved folder.


Expected results:

It should show the previously saved folder. For example, if I save a bookmark in a "Work stuff" folder. The next time I bookmark another page should have "Work stuff" folder as the default. Then if I want to change to another folder, I would click the dropdown menu and select another folder.

This is a new bug that just appear in v58 when I updated Firefox today. It wasn't there in v57.
Component: Untriaged → Bookmarks & History
A related change is surely bug 1120110, but that should only have changed the default destination from the Bookmarks menu to Other Bookmarks, so that clicking the Star or CTRL+D end up doing the same thing.
We store the recently used folders in the dialog popup, but I don't remember that we'd pre-select one of them, so that should be checked.

I hope someone can find the time to confirm this regression.
¡Hola!

I'm marking this NEW as https://support.mozilla.org/questions/1201180 describes the same thing (in Spanish).

¡Gracias!
Alex
Status: UNCONFIRMED → NEW
Ever confirmed: true
Thanks!
Priority: -- → P2
This is not a regression, nor a bug.
It always works like this since I started to use Firefox (10 years ago at least).
And I just tested again in v57 with a new profile, it always use "other bookmarks" as default, not previously used folder (on the contrary, that's how Chrome works).

I suspect you have some addons modified the behavior before.
(In reply to fireattack from comment #4)
> It always works like this since I started to use Firefox (10 years ago at least).
...
> I suspect you have some addons modified the behavior before.

I have also been using Firefox for many, many years, and this is new behavior, introduced within the last week.

I have no Bookmark add-ons installed, and have happily been adding bookmarks CTRL+D'ing for years, and they have always been placed in "Bookmarks Menu" by default, not in "Other Bookmarks".
(In reply to resa from comment #5)
> I have no Bookmark add-ons installed, and have happily been adding bookmarks
> CTRL+D'ing for years, and they have always been placed in "Bookmarks Menu"
> by default, not in "Other Bookmarks".

Not going to bookmarks menu IS NOT a bug, that's intended behavior, both ctrl+d and the star should go to unfiled.

What we are discussing here is whether the bookmarks were going to the last used folder (not bookmarks menu) instead, I didn't have time to test older versions yet.
Sure, sorry if it wasn't clear. I just used "Bookmark Menu" as an example, because I always save new bookmarks to that folder. Anyways, Firefox should remember where last bookmark was saved, and suggest using that folder next time, in stead of suggesting "Other Bookmarks", as it does in the new version.
I just tested again with 56 and 57, and can confirm the behavior is indeed the same as what Marco said.

Click star: 
Always by default add to "other bookmarks", in 56,57,58,Nightly

Use Ctrl+D: 
It used to go "bookmark menu" in 56, 57 (but NOT "previously saved folder"). Now it goes to "other bookmarks" after 58, to be consistent with clicking star (bug 1120110).


So far, it works as intended. If you want to request feature to use the "previously saved folder", which is a function Firefox never had (the "Chrome way", but I would like to have it too), should open a new, more clear ticket for that.
Hello

I also would like to have the previous behavior back (defaulting to bookmarks menu) or - as always - the best solution: let me choose the default location, if you like to keep the "others bookmarks" folder as default. Defaulting to a folder which is designed to be a gathering place for unsorted bookmarks is not the ideal behavior for a modern browser, I think. Especially since there already is a common saving space for bookmarks that are not specifically sorted into subfolders, the bookmarks menu folder. And considering this is a recent change from the previous default which many of us have used for years and thousands of bookmarks. If there are resources available and you want to be over the top (...Chrome?...) you could even think about using the most recent folder as default or predict the to-be folder location in some way.

Also, I think you should reconsider your choice of hiding comments only because they are irksome to you.
Comments are not being collapsed (note they are not removed, for transparency and honesty reasons) for malicious reasons, this is a technical bug tracker, it's not a discussion forum, nor a place to point out our own preferences. This is a workplace for most of us, as such we should stay on topic and everyone should behave properly.

(In reply to you from comment #12)
>let me choose the
> default location, if you like to keep the "others bookmarks" folder as
> default.

I agree, indeed bug 1412263 is open for the community to pick up. It should be easy to change bookmarkPage to read the default guid from an hidden pref.

Regarding this bug, I'll test the behavior with previous versions regarding recently used folders, and figure out if it's an effective bug.
Flags: needinfo?(mak77)
So, I tested Firefox 57, CTRL+D goes to the Bookmarks Menu, clicking the star goes to Other Bookmarks.
In Firefox 58 both go to Other Bookmarks, as expected after bug 1120110.

I couldn't find a way to make it bookmarks to the previously selected folder.

As such, this bug is not a regression, it's an enh request.

You're free to follow (and suggest patches obviously) bug 1412263 to allow to set the default destination folder. I suspect it will be an about:config pref at the beginning, we need UX involvement to do more than that.

If anyone should ever implement this "store in the last folder" behavior (if it's indeed what Chrome does), it could likely reuse the pref added in bug 1412263, that likely would just contain a guid.
Severity: normal → enhancement
Flags: needinfo?(mak77)
Priority: P2 → P5
Personally, I think saving new bookmarks to Bookmarks Menu by default should be the main priority, whereas remembering the previously selected folder is not that important, and maybe not even useful.
Just chiming in here as an end user:

Don't care about Save to Previous Folder.
Would like everything to go initially to Bookmarks Menu (where I can sort it's final destination from there)...
See Also: → 1435545
Depends on: 1412263
Summary: Saving bookmark keep defaulting to "Other bookmarks" folder → Allow the star panel to select the last used folder when bookmarking
This has just started happening to me too. All of a sudden my bookmarks are now defaulting to "Other Bookmarks" whereas they always have defaulted to the bottom of the Bookmarks menu in the past. Don't try to say "I've been using Mozilla since XYZ and this is normal" I have been using Mozilla since Netscape 1.0 and then switched to Firefox upon its inception as well. I'm going back 20 years!!! This is not normal. Now it may be a new thing that has been implemented (Which would be a load of bs 'cause if it anit broke dont fix it!') but it has never done this before. This issue needs to be addressed and fixed ASAP! I believe it's a bug.
Oh and here: https://bugzilla.mozilla.org/show_bug.cgi?id=423617 is a comment from 10 years ago stating this:


	Bayle Shanks
	
Comment 3 • 10 years ago

By default, clicking the star places a bookmark in "unsorted bookmarks", while using the "Bookmark this page" menu item places defaults to the "bookmarks menu" folder.
either way it doesn't seem like this will be fixed so I just added the add-on https://addons.mozilla.org/en-US/firefox/addon/default-bookmark-folder/ which fixed this issue.
(In reply to Chris Adler from comment #19)
> either way it doesn't seem like this will be fixed so I just added the
> add-on
> https://addons.mozilla.org/en-US/firefox/addon/default-bookmark-folder/
> which fixed this issue.

Thanks for that , Chris!

Depends on D94879

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Depends on: 1425290

(In reply to Alice0775 White from bug 1674539 comment #6)

I dislike "last used folder" thing... It will be another discoverability issue.
Because I must have forgotten the last used folder. I'll have to look for where it was saved.

I don't understand why. File open / save as dialogs work the same way. Why would it be a discoverability issue?

Flags: needinfo?(alice0775)
Flags: needinfo?(alice0775)

(In reply to :Gijs (he/him) from comment #27)

(In reply to Alice0775 White from bug 1674539 comment #6)

I dislike "last used folder" thing... It will be another discoverability issue.
Because I must have forgotten the last used folder. I'll have to look for where it was saved.

I don't understand why. File open / save as dialogs work the same way. Why would it be a discoverability issue?

So, I checked "Always ask you where to save files" in option settings.

(In reply to Alice0775 White from comment #28)

(In reply to :Gijs (he/him) from comment #27)

(In reply to Alice0775 White from bug 1674539 comment #6)

I dislike "last used folder" thing... It will be another discoverability issue.
Because I must have forgotten the last used folder. I'll have to look for where it was saved.

I don't understand why. File open / save as dialogs work the same way. Why would it be a discoverability issue?

So, I checked "Always ask you where to save files" in option settings.

Sure, but the starting location of that dialog remembers the last-used place. Bookmarks would be the same - you can change where the bookmarks go, just that the default would be wherever you put the last bookmark, instead of always reverting back to a fixed folder.

(In reply to :Gijs (he/him) from comment #29)

(In reply to Alice0775 White from comment #28)

(In reply to :Gijs (he/him) from comment #27)

(In reply to Alice0775 White from bug 1674539 comment #6)

I dislike "last used folder" thing... It will be another discoverability issue.
Because I must have forgotten the last used folder. I'll have to look for where it was saved.

I don't understand why. File open / save as dialogs work the same way. Why would it be a discoverability issue?

So, I checked "Always ask you where to save files" in option settings.

Sure, but the starting location of that dialog remembers the last-used place. Bookmarks would be the same - you can change where the bookmarks go, just that the default would be wherever you put the last bookmark, instead of always reverting back to a fixed folder.

Okay. I understand.

In case of uncheck "Show editor when saving", how does "last used folder" treat?.

Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/455ff5647429
use the last-used folder for new bookmarks, r=mak
Flags: needinfo?(gijskruitbosch+bugs)
Flags: needinfo?(gijskruitbosch+bugs)
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/58edeb3eb4e0
use the last-used folder for new bookmarks, r=mak
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
Blocks: 1412263
No longer depends on: 1412263

(In reply to Alice0775 White from comment #30)

In case of uncheck "Show editor when saving", how does "last used folder" treat?.

It behaves the same way - ie it still remembers things. We discussed this with our UX folks and decided to keep this behaviour like this for now and see what people do with it. It probably isn't a problem unless people both opt out of showing the editor when saving, and use the same editor to change the location of bookmarks (rather than the sidebar or the library, which don't alter the pref). That seems like an unlikely combination.

It's also not clear what would be better - it'd be unexpected for most users if we start updating the folder if you tick that checkbox and then stop if you untick it. It'd also be unfortunate to just never update, or to get rid of the option not to show the editor when creating bookmarks. We considered not updating the default folder if you edit bookmarks with the dialog (as opposed to creating new ones), but that also creates a strange inconsistency, and also doesn't match what other browsers do so would confuse users who are used to that behaviour.

So for now, we're going to try it like this. Happy to hear if there is some other solution we're missing here or a usecase we're not aware of, that should trump all the other concerns at play...

This was working for a week or two but is no longer working as of 84.0b7 (or maybe an earlier version, not sure exactly when it stopped working). I opened a new bug because I wasn't sure whether to reply here or open a new one.

https://bugzilla.mozilla.org/show_bug.cgi?id=1680337

(In reply to vaindil from comment #36)

This was working for a week or two but is no longer working as of 84.0b7 (or maybe an earlier version, not sure exactly when it stopped working). I opened a new bug because I wasn't sure whether to reply here or open a new one.

https://bugzilla.mozilla.org/show_bug.cgi?id=1680337

Yes, the fix for this bug was disabled in late beta. We will do some testing in the 84 release cycle and then hopefully roll out the changes to everyone in 85 or 86.

Summary: Allow the star panel to select the last used folder when bookmarking → Allow the star panel to select the last used folder when bookmarking (fixed in early beta and earlier, behind browser.toolbars.bookmarks.2h2020)
No longer regressions: 1682488
QA Whiteboard: [qa-85b-p2]

Thanks for fixing this. Now (Firefox 84.0.2) browser.bookmarks.defaultLocation under about:config is updated, according to where I last created a bookmark by pressing CTRL+D. For example the value for Bookmarks Menu is menu________, whereas the value for a customly named folder is set to something like oJ7SkpfTj_6J.

The title could be updated, and CTRL+D added, since this not only affects bookmarking by clicking the star icon?

Summary: Allow the star panel to select the last used folder when bookmarking (fixed in early beta and earlier, behind browser.toolbars.bookmarks.2h2020) → Remember the last used folder when bookmarking through Firefox's UI

This change broke the single-click bookmarking feature. Before, you could click the Star to remember a site and find it again by going to "Other Bookmarks". Now after you click the star the bookmark is saved to a "random" folder (wherever you last saved a bookmark) and can only be found again by searching your bookmarks. Obviously you can manually adjust the folder, if you remember to override the habit of a decade, but single-click Bookmark saving was one of the most convenient features of the Firefox Bookmark system IMO.

p.s. I'm happy to file a bug if that is more appropriate than commenting here.

I don't get your issue.

All you need to to is to change the save folder to "Other Bookmarks" for the very first bookmark you saved.

Then, all the future saved bookmarks would be saved in this folder by default, because it is your "last saved" folder.

(In reply to PikeUK from comment #39)

This change broke the single-click bookmarking feature. Before, you could click the Star to remember a site and find it again by going to "Other Bookmarks". Now after you click the star the bookmark is saved to a "random" folder (wherever you last saved a bookmark) and can only be found again by searching your bookmarks. Obviously you can manually adjust the folder, if you remember to override the habit of a decade, but single-click Bookmark saving was one of the most convenient features of the Firefox Bookmark system IMO.

p.s. I'm happy to file a bug if that is more appropriate than commenting here.

Please file a new bug with more specifics - comment 40 is on-point in that it is not clear to me off-hand why the bookmark would get saved to a "random" folder - if the previous bookmark you saved was saved in "other bookmarks", that's where the next one should be saved to, too, and you shouldn't have any issues. (If not, perhaps there's a bug. But for sure we would need more specifics about what you're doing and what you're seeing Firefox do, to comment in more detail and/or fix anything.)

(In reply to :Gijs (he/him) from comment #41)

(In reply to PikeUK from comment #39)

This change broke the single-click bookmarking feature. Before, you could click the Star to remember a site and find it again by going to "Other Bookmarks". Now after you click the star the bookmark is saved to a "random" folder (wherever you last saved a bookmark) and can only be found again by searching your bookmarks. Obviously you can manually adjust the folder, if you remember to override the habit of a decade, but single-click Bookmark saving was one of the most convenient features of the Firefox Bookmark system IMO.

p.s. I'm happy to file a bug if that is more appropriate than commenting here.

Please file a new bug with more specifics - comment 40 is on-point in that it is not clear to me off-hand why the bookmark would get saved to a "random" folder - if the previous bookmark you saved was saved in "other bookmarks", that's where the next one should be saved to, too, and you shouldn't have any issues. (If not, perhaps there's a bug. But for sure we would need more specifics about what you're doing and what you're seeing Firefox do, to comment in more detail and/or fix anything.)

Comment 40 would only make sense if you never use the dropdown to file bookmarks into different folders and only ever rely on single-click bookmarking. I've filed Bug 1691729 with more details and explanation.

Regressions: 1691729
Regressions: 1748345
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: