-
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
Basic Tests for Q, W, Shift+Q, Shift+W #3104
Conversation
Tests for basic functionally. Going between 128th duration & whole duration and applies commands 'half-duration', 'double-duration', 'dec-duration-dotted', 'inc-duration-dotted' Class Score gets methods cmdDoubleDuration(), cmdHalfDuration(), cmdIncDurationDotted(), cmdDecDurationDotted() which are public wrappers around private method cmdIncDecDurationDotted. Simplified the text description of the commands
{ "double-duration", [this]{ cmdDoubleDuration(); }}, | ||
{ "half-duration", [this]{ cmdHalfDuration(); }}, | ||
{ "inc-duration-dotted", [this]{ cmdIncDurationDotted(); }}, | ||
{ "dec-duration-dotted", [this]{ cmdDecDurationDotted(); }}, |
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.
I think these wrapper functions make code easier to read.
void cmdDoubleDuration() { cmdIncDecDuration(-1, 0); } | ||
void cmdHalfDuration() { cmdIncDecDuration( 1, 0); } | ||
void cmdIncDurationDotted() { cmdIncDecDuration(-1, 1); } | ||
void cmdDecDurationDotted() { cmdIncDecDuration( 1, 1); } |
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.
public wrapper functions around the private function cmdIncDecDuration
QT_TRANSLATE_NOOP("action","Decrease Duration to Next Smaller Dotted or Non-Dotted Duration"), | ||
QT_TRANSLATE_NOOP("action","Decrease duration to next smaller dotted or non-dotted duration"), | ||
QT_TRANSLATE_NOOP("action","Decrease Duration Dotted"), | ||
QT_TRANSLATE_NOOP("action","Decrease duration dotted"), |
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 command's description has been simplified, for reasons I mentioned in: https://musescore.org/en/node/175461#comment-682611
note this should go in both master and 2.1, I think. |
2.1 is too different to merge this automatically. Would you mind creating a new PR ? |
ok, I'll create a new PR. |
Tests for basic functionally. Going between 128th duration & whole duration and applies commands 'half-duration', 'double-duration', 'dec-duration-dotted', 'inc-duration-dotted'
Class Score gets methods cmdDoubleDuration(), cmdHalfDuration(), cmdIncDurationDotted(), cmdDecDurationDotted() which are public wrappers around private method cmdIncDecDurationDotted.
Simplified the text description of the commands