-
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
enhance text tab command to preserve placement #5475
Conversation
b4f441b
to
b2c88ec
Compare
This should work for other text types as well, BTW. I call it "fingering mode" but usually put it in quotes, because it actually works for staff text etc. Only difference is you need Alt+Right/Left rather than Space/Shift+Space to do the navigation. So this change should in theory also allow you to flip a staff text below, start editing it, and hit Alt+Right and the new texts are also below. I haven't built this but I expect it works as expected :-) |
b2c88ec
to
f9d9436
Compare
@MarcSabatella I included staff text & system text as well. Thanks for clarifying! I didn't even know they can do the same. I'm just not sure if I called it right, text-tab mode? Is there a more conventional name for that? |
Since the original impetus was for fingerings and only during implementation did I see that it could work just as easily for other text, “fingering mode” is still the most recognized name. But it’s not just for fingering and it’s not a separate mode at all. So, “text tab command” is probably accurate. |
f9d9436
to
ccb41fb
Compare
Done, thanks! |
I'm interested in script test. I'll see if I can create one for this. But I see in that introduction forum topic, "Text edit is not recorded", so I'm not sure whether a script test can be created for this... |
ccb41fb
to
3be8964
Compare
Well, text edit isn't recorded, but adding texts is. |
3be8964
to
ef51bc3
Compare
ef51bc3
to
0f24dfd
Compare
I don't understand, why did Travis build break off? |
The build just took too long, for some reason. Most likely not your fault and restarting that Travis now would most probably pass |
I did it twice but got the same error, and the build stopped exactly when the scripts were build. Doesn't look like a coincidence. |
Hmm, then that script might be the issue |
But what kind of issue exactly can block a build?! Infinite loop? There isn't any. |
0f24dfd
to
3ed997c
Compare
You can try reproducing this error by running this script test from a command line:
However text editing is indeed currently not recorded by the script recorder engine, and |
3ed997c
to
2ba1f27
Compare
Deleted the tests. |
Using style settings seems to be a semantically correct way to put all texts above/below staves and it gives a lot of benefits for further score editing. Would it perhaps make sense to limit this enhancement only to fingerings, or maybe make it possible to define an extra fingering style (which would create compatibility issues though)? As far as I understand this change would make it easier to accidentally create large numbers of text elements that do not follow style settings which I would try to avoid if possible. |
As far as I understand, this PR only deals with text tab command, and it's basically not possible to have this kind of "accidents" given that you're already using text tab command. If you insert a unstyled text and then use text tab command, you most probably do want to make all subsequent texts have the same placement. |
Resolves: https://musescore.org/en/node/297092#comment-958921. If current text (staff text/system text/fingering) is `UNSTYLED` or `NOSTYLE`, the next text created using text tab command inherits the same `PropertyFlags` and `Placement`, making it much easier to put all texts above/below staves.
2ba1f27
to
ed5ed8f
Compare
Resolves: https://musescore.org/en/node/297092#comment-958921.
If current text (staff text/system text/fingering) is
UNSTYLED
orNOSTYLE
, the next text created using text tab command inherits the samePropertyFlags
andPlacement
, making it much easier to put all texts above/below staves.