-
Notifications
You must be signed in to change notification settings - Fork 27
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
2.0.1 current focus is not highlighted in bank #228
Comments
Hm, I am unsure about this one, because the patch could be in the bank multiple times (e.g. if you make yourself an init patch and then drag that into a new bank, the whole bank is filled). Should all light up then? Maybe. |
So that being said, this actually wasn't even meant to be the case, so I'll flag this an enhancement rather than a bug, |
You can see U1 P13 is slightly highlighted in the bank, and that is just as I expect it, because I clicked on a patch in a bank panel. But it should be clearly visible. That's all there is The case you are referring to is probably about something different: what should happen when the user clicks on a patch in the library, and the same patch is used multiple times in the bank. I'd handle this the same as the favorites, all occurrences should be highlighted, but that should look different than the highlight when somebody directly selects a patch. Thinking about this I may have found a way to handle program changes and edit buffer dumps consistently without a switch. Add two clickable areas, one for the program number, which sends a program change, and one for the patch name, which sends the edit buffer. Now the user can choose what they want in each case. That would work both in the bank panel and in the library, but in the bank panel I guess it's better to split the button horizontally between patch number and patch name. |
Ah, right, the focus colour is off and hardly visible. That's indeed a bug! |
So I made a fix that all patch buttons highlight when they have the same patch (md5 sum) as the current patch, which is the one loaded into the currentpatch panel. This seems to work nicely for the bank as well as for the second window. |
Fixed in 2.1.0 |
I mean, it is, but ever so slightly. In the grid, the highlight is much more obvious. Should be the same color in the bank panel.
The text was updated successfully, but these errors were encountered: