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

optional removal of custo before <eu> blocks #761

Closed
eroux opened this issue Jan 7, 2016 · 6 comments
Closed

optional removal of custo before <eu> blocks #761

eroux opened this issue Jan 7, 2016 · 6 comments
Assignees
Milestone

Comments

@eroux
Copy link
Contributor

eroux commented Jan 7, 2016

An abbey asked for the possibility to prevent custo to appear if there is a line break before <eu> blocks

@eroux eroux added this to the 4.1 milestone Jan 7, 2016
@henryso
Copy link
Contributor

henryso commented Jan 7, 2016

Markdown ate the <eu> in the description.

@eroux
Copy link
Contributor Author

eroux commented Jan 7, 2016

indeed, thanks!

@henryso
Copy link
Contributor

henryso commented Jan 7, 2016

I tried the obvious change of putting \gre@usemanualcustos at the start of \GreBeginEUOUAE (unconditionally, just to see if it would work), but it didn't work. What am I missing? Is there some TeX magic going on here?

@eroux
Copy link
Contributor Author

eroux commented Jan 7, 2016

Why not something like \greblockcustos? Also, I'm affraid that in some cases, you'll have to tell the syllable before the <eu> block not to put some custo, because the localrightbox will already typeset at \greBeginEUOUAE, but I'm not 100% sure...

@henryso
Copy link
Contributor

henryso commented Jan 7, 2016

If I'm reading this correctly, \gre@usemanualcustos sets \ifgre@blockeolcustos to true and clears the localrightbox, so it's probably the prior syllable thing.

@henryso
Copy link
Contributor

henryso commented Jan 7, 2016

You were right, localrightbox needs to be cleared on the prior syllable.

@henryso henryso self-assigned this Jan 7, 2016
henryso added a commit to henryso/gregorio that referenced this issue Jan 7, 2016
henryso added a commit to henryso/gregorio-test that referenced this issue Jan 7, 2016
eroux added a commit to gregorio-project/gregorio-test that referenced this issue Jan 8, 2016
@eroux eroux closed this as completed Jan 8, 2016
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