-
Notifications
You must be signed in to change notification settings - Fork 43
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
Finer control on notes-text spacing #1125
Comments
There is a request to try to get this specifically into 5.0, so if everyone else is busy, maybe someone can give me pointers to "what's happening at the top of the staff", so I can try to look into this? |
I think the important description is this one. What I remember is that you can define a threshold above which notes will start to move the space above staff lines, if you set it high, no note will change the above staff line space, whatever its height; if you set it to 0, this means that |
Should the space be based on the number of notes before the threshold (e.g., |
First option. Also, it would be great if the variables can be tuned per-line. there are a few cases in the italian Antiphonale where the custos is relatively far from the text (horizontally) but is the only very low note |
Note: while I am looking into this, things are slow going as I experiment with stuff. If anyone has the time and inclination to take this on, please feel free. |
What should the default be? "all notes below c will add 2mm"? |
I think it should be all notes below c will add half of the space between staff lines... |
Maybe it's because I'm misunderstanding what should be done, but given the other spaces, I think half the space between staff lines (which is almost 3 mm with a grefactor of 17) is rather excessive. In this screenshot, the lowest pitch on the top staff is Is this what is desired? |
Hmmm, it looks like you're modifying the wrong space here... the space that should be set is the space between the baseline of the text and the bottom staff line (the one on In your pictures, the space between lines is 10px, half is 5px. The base value of the space according to the second line is 23px. So this means that on the first line, the space should be 23+3x5=38px, while in your image it's 57px... do you see what I mean? |
Yes, looking at these pictures this way, it's more obvious that I'm doing something wrong. I'll keep banging my head against it. |
I think the problem is that the modification of the space looks applied twice: in your second post, when you disable the feature, it seems the space is modified anyway... ? |
No, when disabled, that picture matches the old expectation from that test. Looking at the code, I think I am double counting |
Hmm, yes, but that feature should replace the current modification of the space, not add another modification on top of it! |
it's just a more convenient way to tune this spacing (and it's necessary for correctly tuning it), it's not really something new... |
Ok, that was totally unclear from the opening post. I'll see if I can figure out how to do that. |
Sorry, I didn't realize it could be interpreted another way! Basically I think the calculation already more or less does this (add half the space between staff lines for each note beneath d), the feature is just to add a way to tune the threshold and the value |
Ok, then the current calculation is 15750 sp * the number of pitches below c, which I guess this shall be the default. And this issue is actually requesting that "15750 sp" and "the number of pitches below c" be configurable, correct? |
yes |
Enhancement mentioned in gregorio-project#1125.
Part of implementation for gregorio-project#1125.
In some cases, a short
spacelinestext
should maked
pitch considered as a low note adding space between notes and text.See the following example
in which the
i
merges within the podatus. Of course this example is a bit extreme, but I think having a system similar to what's happening at the top of the staff with (additionaltopspacealtthreshold
) would fix this issue...The text was updated successfully, but these errors were encountered: