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

spaceafterlineclef@close #1138

Closed
eroux opened this issue Jun 14, 2016 · 10 comments
Closed

spaceafterlineclef@close #1138

eroux opened this issue Jun 14, 2016 · 10 comments
Milestone

Comments

@eroux
Copy link
Contributor

eroux commented Jun 14, 2016

It would be useful to differenciate spaceafterlineclef when the notes are "close" to the key or not, for instance in (c4) (a) the note is far, and in (c3) (h) the note is closed. There are a few corner cases (the second note should be taken in the case of a pes), but it can be made in a very simple way.

@henryso henryso added this to the 4.2 milestone Jun 14, 2016
@henryso
Copy link
Contributor

henryso commented Jun 14, 2016

Can you define how close is close? For example (c4) physically spans (i) through (k), so would (h) and (l) be considered close? How about (g) or (m)? I'm assuming in the case of the pes that "close" applies if either note is close, correct?

@eroux
Copy link
Contributor Author

eroux commented Jun 14, 2016

I think h and g would be considered close yes, but f not... You're right about the pes

@henryso henryso self-assigned this Jun 14, 2016
@henryso
Copy link
Contributor

henryso commented Jun 14, 2016

The way I see to implement this, we need to redefine the localleftbox before every glyph. Is there a better way?

@eroux
Copy link
Contributor Author

eroux commented Jun 14, 2016

Wow, it seems I spoke too fast when I said that it would be easy to implement... I think using the bolshift would be easier, but it's not trivial neither... we can delay that one for later... The real blocking issues for the Antiphonale project are #1141 and #1137, if the others are complex, we can delay them...

@henryso henryso removed their assignment Jun 14, 2016
@eroux
Copy link
Contributor Author

eroux commented Jun 15, 2016

Thinking about it, this feature poses another big problem: the left-alignment of lines on notes will be broken... The real-life example of this feature was actually for the very first note of a score, after an initial. I think it would be fairly easy to handle this case only, and close this ticket (as breaking the left-alignment looks like a bad idea)... What do you think?

@henryso
Copy link
Contributor

henryso commented Jun 15, 2016

In that case, it seems like it would only be applicable for one-line initials, as applying it to the first line of no-initial and two-line-initial scores would break left alignment in the same way.

@eroux
Copy link
Contributor Author

eroux commented Jun 15, 2016

indeed!

@eroux
Copy link
Contributor Author

eroux commented Jun 26, 2016

I think this one is the most important of the remaining features for 4.2, I think I can make the TeX part, the C part would consist only in adding an argument to \gresetinitialclef which would be 0 or 1 if the clef and notes can be considered close or not... I think we can just use 0 where there are two clefs or a flat on the clef. @henryso do you think you could make the C part in a branch if you have time?

@henryso
Copy link
Contributor

henryso commented Jun 26, 2016

@eroux I will try to work on the C part of this tonight.

@henryso
Copy link
Contributor

henryso commented Jun 26, 2016

I created the fix-1138 branch with the changes you requested. The gabc-gtex tests fail (of course), but no other tests change. I didn't create a branch in the test repository as I'm sure you'll be doing that after this is completed.

eroux added a commit that referenced this issue Jun 27, 2016
eroux added a commit that referenced this issue Jun 27, 2016
@eroux eroux mentioned this issue Jun 27, 2016
@eroux eroux closed this as completed Jun 28, 2016
rpspringuel added a commit to rpspringuel/gregorio that referenced this issue Jul 4, 2016
* commit 'f790c50299dcbf2d919e4459de8668ce6c269b19': (35 commits)
  Specifically noted which spaces may be altered by grechangenextscorelinedim.
  Added the ability to change a dimension for a given line in a score. Fixes gregorio-project#1156.
  1144: fix doc typos
  oops
  document spaces
  doc, fix small bug,
  Implemented distance check for notes around virgula and divisio minima.
  do not use alterationsadjustment on clef change, reduce it
  greciliae: fix small problem of joining line height
  doc
  fix gregorio-project#1169
  attempt to fix gregorio-project#1146
  TeX part of gregorio-project#1144
  Extended the nearness check to the first two notes of podatus, clivis, and porrectus.
  gregorio-project#1138: TeX and doc
  gregorio-project#1138: change default case
  gregorio-project#1137: fix case of podatus
  Added an argument to \GreSetInitialClef indicating whether the first note is near the clef. Part of the implementation for gregorio-project#1138.
  fix gregorio-project#1137
  Add documentation for recent Nabc vertical placement changes.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants