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

FR: "SWS: Select previous folder" and "SWS: Select next folder" to be a little smarter #981

Closed
ScottChesworth opened this issue May 15, 2018 · 4 comments

Comments

@ScottChesworth
Copy link

@ScottChesworth ScottChesworth commented May 15, 2018

Hi,

Currently, these two actions only work when a folder track is selected. Would it be possible to modify them so that if a user is on a standard track when they trigger the actions, they land on the nearest folder in either direction?

Example: let's say I have 12 drum tracks contained within a folder called "Drum bus", followed by two bass tracks which aren't folders, and below those, let's say 6 guitar tracks contained within a folder called "Guitar bus".

Currently, to navigate between those two folders using the SWS actions, I have to have one of the existing folders selected first. IE, if my snare track is selected, it's not possible to hit the keystroke I've bound to "SWS: Select next folder" and move to the "Guitar bus" folder.

This change would make folder navigation way more intuitive, and be massively appreciated by a whole bunch of blind users that I'm involved in putting a keymap together for.

Thanks for considering,

Scott

@nofishonfriday
Copy link
Collaborator

@nofishonfriday nofishonfriday commented May 15, 2018

Hi,

modifying existing actions is always a little problematic imo because of the risk of breaking existing workflows.

So I've added two new actions for the next SWS version called "SWS: Select nearest previous folder" and "SWS: Select nearest next folder" which behave the way you're suggesting.

nofishonfriday added a commit to nofishonfriday/sws that referenced this issue May 15, 2018
@ScottChesworth
Copy link
Author

@ScottChesworth ScottChesworth commented May 17, 2018

@ScottChesworth
Copy link
Author

@ScottChesworth ScottChesworth commented Jun 16, 2018

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

Successfully merging a pull request may close this issue.

None yet
2 participants