Giter Club home page Giter Club logo

Comments (23)

ryanwwest avatar ryanwwest commented on August 22, 2024 1

Interesting, ctrl-tab seems to work for moving sidepanes forward, but ctrl-shift-tab doesn't move them in reverse (though it does for the root/main tabs). I image a future Obsidian update would fix this, but hard to say when. I'm not sure what option is better, interested to hear what @FelipeRearden thinks.

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024 1

Thanks for the testing! I forgot to mention that you still need to enable the different view types. So basically call for every view type in the sidebar you want to cycle the "Enable this view type" command.
I won't find the time to fix the search focus in the next days. So it will take just a bit more time.

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

Do I understand you correctly, that you want 4 new commands. 2 for each side to go to the left/right tab inside the sidebar?

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

That's right, I haven't thought of anything more intuitive.

While the link I added also talks about additional commands to focus on each side pane (no matter which tabis currently active) or the main pane, that's probably outside the scope of this plugin.

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

Well, I don't think an extra command just to change focus on the sidebar is needed. I could implement it that way the switching command can be used to focus on either sidebar and on next call actually switch to left or right.

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

Are you saying if you have a ~go to next left sidebar tab command, then the first invocation focuses on the pane, but doesn't actually switch it - and then the second invocation does advance the tab to the right?

That seems a bit confusing to me... but possibly. Do other programs have a similar behavior?

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

I guess another way to do it is use the same, existing next/previous commands for the main tab but have them apply to the active portion of the window. So have some sort of commands to move the focus between the main pane (or if there are multiple main panes open, between each of them), the left sidebar, and the right sidebar, and the next/previous tab commands apply to whatever the focus is currently on.

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

Hello @Vinzent03 and @ryanwwest

Trying to help here ....

Just to let you guys know that there is a plugin called Cycle in Sidebars maybe it will help you to achieve what you want πŸ™

I hope it helps :)

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

Thanks @FelipeRearden ! This is the same thing that I originally suggested and appears to work well. @Vinzent03 , I still think this FR (or the final variation of it I mentioned that uses the same hotkeys for cycle forwards/reverse in any pane, mains or left/right sidebar according to focus) could fit in well in the scope of the plugin and thus provide more value overall than that one. However, it's up to you since there's a good alternative now - just as fine if this is closed because a solution exists.

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

Actually, I just realized that @FelipeRearden opened a very similar varied FR of this on that other plugin: houcheng/obsidian-cycle-in-sidebar-plugin#4, and another regarding moving focus to the sidebar: houcheng/obsidian-cycle-in-sidebar-plugin#5. So he has pretty much my same thought process haha. I realized that you can already use the default Obsidian commands Focus on tab group above/below/to the left/to the right to also move from the sidebar focus into the main focus, so we really just need a command to get into each sidebar (preferably, the Obsidian devs would just let these same commands also move us into sidebar focus).

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

So we need two commands to focus on either sidebar. To cycle in them, we could use the existing ones for the root tabs?

from tab-switcher.

ryanwwest avatar ryanwwest commented on August 22, 2024

Yeah, what if we use the ones you already came up with to focus on whatever pane / sidebar pane is currently focused/active (like houcheng/obsidian-cycle-in-sidebar-plugin#4)? Then the only thing left is a way to focus on left or right sidebar as focusing back on the main panes is already solved.

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

So we need two commands to focus on either sidebar. To cycle in them, we could use the existing ones for the root tabs?

Exactly @Vinzent03 !!!! What is missing to close the workflow is two commands:

1️⃣ focus on left sidebar
2️⃣ focus on right sidebar

Because we already have a way to focus on active note. What we dont have today is a way to focus on the sidebars.

Yeah, what if we use the ones you already came up with to focus on whatever pane / sidebar pane is currently focused/active (like houcheng/obsidian-cycle-in-sidebar-plugin#4)? Then the only thing left is a way to focus on left or right sidebar as focusing back on the main panes is already solved.

100% agree with @ryanwwest comment :)

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

So do you want to use the commands provided by this plugin to cycle through the panes within the sidebar or the ones by Obsidian itself. Currently, the commands from this plugin only work with the root tabs. The ones from Obsidian itself already work within the sidebar.

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

So do you want to use the commands provided by this plugin to cycle through the panes within the sidebar or the ones by Obsidian itself. Currently, the commands from this plugin only work with the root tabs. The ones from Obsidian itself already work within the sidebar.

I have a solid opinion about this new feature but I am gonna wait for the @ryanwwest reply since he is the creator of the FR :)

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

I'm not sure what option is better, interested to hear what @FelipeRearden thinks.

@Vinzent03 let me share my opinion about this ...

1️⃣ The pain point is that we can NOT cycle through the sidebars using the keyboard.
2️⃣ In my humble opinion doesn't make sense to cycle through PANES + SIDEBARS . We need a way to cycle ONLY through SIDEBARS and ONLY through PANES

Based on this have have these new features:

πŸ’» New Command Cycle through Left Sidebar
πŸ’» New Command Cycle through Right Sidebar

πŸ†˜ BUT these new commands need to have a special feature !!!!!!!!!!!!
β†’ While cycle through the sidebars it need to KEEP the focus on the sidebars instead of jump to the active note (app.commands.executeCommandById('editor:focus')) every time we cycle inside the sidebar.

β†’ in others words we need to focus on the active sidebar while we are performing the cycle command


Let me know what you guys think about this and sorry again for step into the conversation πŸ™

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

What about using the existing commands for this case. So they cycle within the current split of the window. Meaning, if you are already in a sidebar, it cycles within that. If you are already in the main tabs, everything stays the same.

By the way, using Go to next tab and Go to previous tab from core Obsidian in the sidebar works great for me. Though, note that I'm on the latest Insider version.

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

What about using the existing commands for this case. So they cycle within the current split of the window. Meaning, if you are already in a sidebar, it cycles within that. If you are already in the main tabs, everything stays the same.

You are a legend @Vinzent03 πŸ₯‡πŸ₯‡πŸ₯‡ spectacular idea to make the commands dynamic !!!! it's the best scenario :)

Please please please make sure that we don't lose the focus that already is in the sidebar while cycle πŸ™πŸ™πŸ™

By the way, using Go to next tab and Go to previous tab from core Obsidian in the sidebar works great for me. Though, note that I'm on the latest Insider version.

Just to let you know ....

πŸ“Έ In the latest public release the commands doesn't show in command pallette WHEN the focus is on the sidebar .
BD4E2CB0-DBD2-4002-8672-B443DD779CF0

⬆️The yellow arrow shows that the focus is in the sidebar instead of the open note

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

Should be done now! πŸŽ‰ There are now two new commands to focus on the sidebar. The option (by default enabled) to stay in the current split.

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

Should be done now! πŸŽ‰ There are now two new commands to focus on the sidebar. The option (by default enabled) to stay in the current split.

Hello @Vinzent03 and @ryanwwest !!!

I tested the new release and I wanna share some information with you :)

My Environment

πŸ’» Obsidian Latest Public Release

⌨️ I tested on Desktop and Mobile (iPad with mouse and keyboard)

πŸ“Ή Screen record is on iPad but the behavior is the same on Desktop

1️⃣ Left Sidebar with 3 Core Plugins: File Explorer, Tags, Search and Starred

Tests

I tested two workflows:

πŸ…°οΈ Focus on Left Sidebar β†’ From Active Note to Sidebar (open and/or closed)

πŸ…±οΈ Cycle Through Panes β†’ Cycle Between Sidebar items: File Explorer, Tags, Search and Starred

πŸ…°οΈ Focus on Left Sidebar

🐞 I found a issue using the Search Core Plugin when the Search is Empty β†’ it is NOT focusing on the search text field when the search is empty

πŸ“Ή Search Not empty
video

πŸ“Ή Search empty
video

βœ… All good with File Explorer and Tags (Starred doesn't have keyboard navigation)

πŸ…±οΈ Cycle Through Panes β†’ Cycle Between Sidebar items: File Explorer, Tags, Search and Starred

πŸ’» Settings
E4CDF7CE-56AE-42E2-98BA-4213A0F6098E

🐞 Some typo is missing in the end :)

🐞 Unfortunately I couldn't Cycle between the Items using any of the commands
β†’ Cycle Next Tab
β†’ Cycle Previous Tab
β†’ Cycle Right Tab
β†’ Cycle Left Tab

ℹ️ On Desktop I got one Cycle from File Explorer to Tags but nothing more
ℹ️ On Mobile I got nothing

πŸ‘Maybe this is an issue because I am on public release and you are on the new Beta ;

πŸ‘I hope you could reproduce some items on your end πŸ™


Let me know if you need more tests . I am here to help !

Have a great day !

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

I won't find the time to fix the search focus in the next days. So it will take just a bit more time.

No problem at all :) Thanks for let me know, I really appreciate :)

Thanks for the testing! I forgot to mention that you still need to enable the different view types. So basically call for every view type in the sidebar you want to cycle the "Enable this view type" command.

My bad, I should have thought about this before testing !!!!

This way, just to let you know about this feature on Desktop and Mobile

βœ… It's working perfect on DESKTOP exactly that way that I always wanted to navigate through Sidebars

πŸ†˜ It's NOT working on MOBILE unfortunately even after enable these different view types I was not able to cycle between the views on Sidebar . Same behavior when using the mobile emulator on Desktop :(

β†’ I am talking about the behavior Mobile because I use Obsidian Mobile on iPadOS with mouse and keyboard as my main device
β†’ Just to let you know if you wanna take a look in the future πŸ™


Lets keep talking @Vinzent03 and thank you very much for everything !!!!!

from tab-switcher.

Vinzent03 avatar Vinzent03 commented on August 22, 2024

Should be fixed now πŸŽ‰ Mobile sidebar works a bit differently...

from tab-switcher.

FelipeRearden avatar FelipeRearden commented on August 22, 2024

Should be fixed now πŸŽ‰ Mobile sidebar works a bit differently...

Hello @Vinzent03 !!!!

Forgive me for my late reply :)

πŸ…°οΈ Focus on Search Text Field β†’ Working Perfect on version 1.2.1 - Mobile and Desktop βœ…

πŸ…°οΈ Cycle Through Sidebar Panes β†’ Working Perfect on version 1.2.1 - Mobile and Desktop βœ…

Thank you very much for the fix πŸ™

Running my tests, I found a new bug ONLY ON MOBILE ....

πŸ†˜ The Command Focus on Left Sidebar stopped working on version only for Mobile (and Desktop emulator)

β†’ By this I mean 2 things:

1️⃣ I CANT set the focus from the Active Note to the Sidebar using the command the same way that I do on version 1.2.0

2️⃣ While I am cycle through the Sidebar, I dont have focus on the Sidebar. When I press the command Focus on Left Sidebar after cycle I CANT get focus as well.

Sorry for bring more problems to you πŸ™

I wish you a fantastic day β˜€οΈ

from tab-switcher.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.