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

2.0.1 current focus is not highlighted in bank #228

Closed
bboc opened this issue Feb 16, 2023 · 6 comments
Closed

2.0.1 current focus is not highlighted in bank #228

bboc opened this issue Feb 16, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@bboc
Copy link

bboc commented Feb 16, 2023

image

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.

@christofmuc
Copy link
Owner

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.

@christofmuc christofmuc added the bug Something isn't working label Feb 18, 2023
@christofmuc
Copy link
Owner

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,

@christofmuc christofmuc added enhancement New feature or request and removed bug Something isn't working labels Feb 18, 2023
@bboc
Copy link
Author

bboc commented Feb 18, 2023

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.

@christofmuc
Copy link
Owner

Ah, right, the focus colour is off and hardly visible. That's indeed a bug!

@christofmuc christofmuc added bug Something isn't working and removed enhancement New feature or request labels Feb 19, 2023
@christofmuc christofmuc changed the title 2.0.1 current patch is not highlighted in bank 2.0.1 current focus is not highlighted in bank Feb 19, 2023
@christofmuc
Copy link
Owner

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.

@christofmuc christofmuc added the Ready for release Will be part of the next release label Oct 14, 2023
@christofmuc
Copy link
Owner

Fixed in 2.1.0

@christofmuc christofmuc removed the Ready for release Will be part of the next release label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants