Skip to content

expand tab actions #1447

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

Closed
wants to merge 1 commit into from
Closed

Conversation

fidgetingbits
Copy link
Collaborator

Similar to #1446 this just expands the available actions for using tabs.

There is one thing I realized which is that here I've used tab flip to indicate flipping back between the most recent tabs. I generally use flip for going to the most recent thing everywhere in my repo: windows, desktops/workspaces, splits, tabs, files, folder, etc. However, community uses split flip to change the horizontal/vertical orientation. Since there is already #1446 open, it might be a good time to change that.

@fidgetingbits
Copy link
Collaborator Author

fidgetingbits commented May 30, 2024

Also I think we should move this to tags/tabs.{py,talon} instead of core/windows_and_tabs to be more consistent, but I won't do it until others weigh in. Not sure why splits is in tags/ and this has it's own special place.

@fidgetingbits
Copy link
Collaborator Author

Since there has been discussion about minimizing forking, and tab is a command I happen to change because I don't find it reliable (I use tabby), I'd also like to propose we make a <user.tab> capture in tabs.py that can be overridden. I'm happy to add it, but I'll wait until others agree.

@fidgetingbits fidgetingbits marked this pull request as draft June 5, 2024 10:59
@fidgetingbits
Copy link
Collaborator Author

After a discussion with @saidelike I will make some more changes before I take it back out of draft.

@fidgetingbits
Copy link
Collaborator Author

Gonna just merge this into a single PR with the split changes since I want to try to unify the names

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

Successfully merging this pull request may close these issues.

1 participant