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

Added support for Dominican-style neumes. #542

Merged
merged 3 commits into from
Jul 11, 2015

Conversation

henryso
Copy link
Contributor

@henryso henryso commented Jul 9, 2015

Implements #1.

This is a bit rough, but it works. I still have a lot of testing to do before this can be merged, but please advise me if you notice anything I should do here.

@eroux
Copy link
Contributor

eroux commented Jul 9, 2015

Seems good to me, but I'm in no position to judge properly...

@henryso
Copy link
Contributor Author

henryso commented Jul 9, 2015

I've run into an issue while testing. A note at d typically doesn't have a long queue (unlike most notes which are on lines), but with the dV> figure in its Dominican form with inverted stem, it runs into the line. I think I can do something with a new TeX command for this particular case, but if anyone has any other ideas, I'm all ears.

@henryso
Copy link
Contributor Author

henryso commented Jul 10, 2015

As far as this change goes, I think this is ready for a final review/merge. If someone familiar with Dominican glyph usage comes forward, we can address their concerns at that time.

@eroux
Copy link
Contributor

eroux commented Jul 10, 2015

ok for me!

@henryso
Copy link
Contributor Author

henryso commented Jul 11, 2015

Hearing nothing to the contrary, I'll merge this.

henryso added a commit that referenced this pull request Jul 11, 2015
Added support for Dominican-style neumes.
@henryso henryso merged commit 99b4ea0 into gregorio-project:develop Jul 11, 2015
@henryso henryso deleted the fix-1 branch July 11, 2015 01:13
henryso added a commit to gregorio-project/gregorio-test that referenced this pull request Jul 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants