-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Guitar bends refinements #19832
Guitar bends refinements #19832
Conversation
b09dfa3
to
5dd7443
Compare
05bfa0a
to
4998cb2
Compare
1845fcb
to
498c673
Compare
Tested on macOS. Everything in the issue description has indeed been rectified. Only minor quibble is that the default content scaling for my palette elements (even after performing a factory reset and resetting the palette) is showing as 1.25 (on macOS), instead of the prescribed scale values in the issue. As such, elements still appear a bit small. This could perhaps be refined later though (so as to not block merging what is otherwise a superb PR). |
sp->appendActionIcon(ActionIconType::PRE_BEND, "pre-bend"); | ||
sp->appendActionIcon(ActionIconType::GRACE_NOTE_BEND, "grace-note-bend"); | ||
sp->appendActionIcon(ActionIconType::SLIGHT_BEND, "slight-bend"); | ||
sp->appendActionIcon(ActionIconType::STANDARD_BEND, "standard-bend", 1.25); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the issue asks for 1.5
sp->appendActionIcon(ActionIconType::GRACE_NOTE_BEND, "grace-note-bend"); | ||
sp->appendActionIcon(ActionIconType::SLIGHT_BEND, "slight-bend"); | ||
sp->appendActionIcon(ActionIconType::STANDARD_BEND, "standard-bend", 1.25); | ||
sp->appendActionIcon(ActionIconType::PRE_BEND, "pre-bend", 1.25); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the issue asks for 1.5
Resolves: #19823