Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Winger forgets "stash home" folder on Firefox restart and creates a new empty one. #54

Open
mkalinski opened this issue Jul 25, 2024 · 2 comments

Comments

@mkalinski
Copy link

To reproduce:

  1. Set up winger to use as "stash home" some named folder within the bookmarks toolbar root. Winger will create this folder.
  2. Move this folder to a different spot on the toolbar (from last position to first, or wherever).
  3. Winger correctly uses this folder as its stash home.
  4. Restart Firefox.
  5. On initialization, Winger creates a new, identically named folder in the original spot, and refuses to use the folder it was using in the previous Firefox session.

In practice, this means that the "stash home" folder can never be moved to a different position than where Winger created it. This is pretty problematic for the bookmarks toolbar.

@mkalinski mkalinski changed the title Winger forgets "stash home" folder on Firefox restart and creates a new empty one. [BUG] Winger forgets "stash home" folder on Firefox restart and creates a new empty one. Jul 25, 2024
@l10nelw
Copy link
Owner

l10nelw commented Jul 25, 2024

I think I know what's going on.

In root folders such as Bookmarks Toolbar, Winger ignores everything before the last separator, if at least one exists (Winger automatically adds a separator if necessary). It's useful if you use a root folder as the stash home, since you'll likely have existing items in it that you want to be statically positioned and not eventually pushed out of view by new stashes.

This feature is currently missing from the documentation, so I need to rectify that. Also this feature isn't very useful if you're not using a root folder as the stash home, so yes I'll regard it as a bug and fix it.

For now, just place your stash home folder after the separator and it should work. Let me know if this isn't the case.

@mkalinski
Copy link
Author

That's actually a pretty useful functionality that would make me organize the stash home differently… had I known it's there. 😛

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants