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

Context Menu: Set Controller To... "sort subfolders into groups of 20", "show CC127" #565

Closed
wants to merge 2 commits into from

Conversation

esaruoho
Copy link
Collaborator

@esaruoho esaruoho commented Feb 10, 2019

this is without all the spacing fixes. easier to review.

this: sorts subfolders by 20 instead of by 10 (less subfolders!)

also makes CC#127 bindable.

52533832-6198b500-2d42-11e9-9eec-e1c38956bfbb

fixes #558

@esaruoho esaruoho changed the title sort in 20, show CC127 Context Menu: Set Controller To... "sort subfolders into groups of 20", "show CC127" Feb 10, 2019
overall this PR closes #558
@baconpaul
Copy link
Collaborator

Lets not merge this and instead just do #566 properly. That ok @esaruoho? The bad UI (which is entirely my fault) isn't much better with 20 terrible things in a group rather than 10!

If you feel strongly happy to merge also.

@esaruoho
Copy link
Collaborator Author

Being able to select like with the dropdownmenu is good stuff but if there is better in the horizon then sure! Nvm the PR then

@esaruoho esaruoho closed this Feb 10, 2019
@esaruoho esaruoho deleted the show127-sort-by-20-with-no-spacingfixes branch February 11, 2019 10:01
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.

Sub-folder says 120-127, but the last bindable CC is 126.
2 participants