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

Unnecessary blank tabs are opened when pinned tabs are present #4

Closed
Leolo1 opened this issue Mar 27, 2024 · 6 comments
Closed

Unnecessary blank tabs are opened when pinned tabs are present #4

Leolo1 opened this issue Mar 27, 2024 · 6 comments

Comments

@Leolo1
Copy link

Leolo1 commented Mar 27, 2024

Hi - I love your idea and visual implementation of the project browser!

When closing a tab (via [x] in the tab bar or via Ctrl.W) which is located to the immediate right of a pinned tab, a (sometimes great) number of empty tabs open. This also happens when opening a note via the project browser on a pinned tab.

Expected behaviour: The tab closes or is opened respectively and no new tabs are opened.

Reproduction A:

  1. Open Obsidian with empty tab showing project browser
  2. Pin this tab
  3. Open any existing note --> a large number of blank tabs open to the right of the tab

Reproduction B:

  1. Open three existing notes
  2. Pin middle tab
  3. Close rightmost tab --> a large number of blank tabs open to the right of the pinned tab
@daledesilva
Copy link
Owner

Thanks for providing such clear reproduction details. I don't pin tabs much so great you were able to catch it.

I'll jump on this in the next day or so and let you know if I can't reproduce.

@daledesilva
Copy link
Owner

I have definitely reproduced it 😳
I'll get to the bottom of it!

@Leolo1
Copy link
Author

Leolo1 commented Mar 28, 2024

I work with a daily note as log book for my activities and pin that one to always have it at hand. That's how I noticed the behaviour. Thank you for taking care of it - happy to help with testing if need be. BTW I am impressed with the roadmap you made for the implementation - seems really well thought out. 👍Personally, I would appreciate to have an option to activate the state menu or not as I see notes to always be in an in-progress status.

@daledesilva
Copy link
Owner

Thanks. The ability to enable and disable (and customise) the state menu will definitely be an option. Just getting around to it.

I was actually considering making it its own plug-in. But probably not.

@daledesilva
Copy link
Owner

This is fixed and in the latest update. Thanks for reporting!

@Leolo1
Copy link
Author

Leolo1 commented Mar 29, 2024

Hi Dale, works like a charm - thank you! 👏

I would support your idea of separating state management from the project browser, as imo an app should solve one problem and that properly. Whoever wants to use state (or whatever other file property) for filtering should be able to do so as I see that indeed to be a valuable feature. But my guess would be that user preferences regarding how to set a file property (via your convenient UI control, the Obsidian-native features or even programatically) will differ. Having your solution as a light-weight extension to manage state which can (but does not have to be) combined with the project browser might be appealing to a lot of users. But that's just me and you should do with your baby whatever the heck you want! Thanks again for your contribution - it's really a great way to navigate the (in my case PARA-structured) vault!

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