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 oriscus cavum glyphs. #727

Merged
merged 4 commits into from
Dec 25, 2015
Merged

Conversation

henryso
Copy link
Contributor

@henryso henryso commented Dec 24, 2015

Fixes #724.

I think the glyphs in greciliae and parmesan are passable, but the shape of the oriscus in gregorio has some irregular control points, so I had a lot of trouble making it look right. Any fixes are welcomed.

Other that this, I think this is ready for review and merge if satisfactory.

All current tests pass, and I updated some tests to include the new figures.

@eroux
Copy link
Contributor

eroux commented Dec 24, 2015

Ok for me, thanks a lot!

@eroux
Copy link
Contributor

eroux commented Dec 24, 2015

In fact, I've just asked the person who asked me, is that ok to wait for her answer? Maybe the lines are too thin... I admit it's a bit strange as they have the same width as others, but they look thinner...

switch (note->u.note.liquescentia) {
case L_AUCTUS_DESCENDENS:
note->u.note.liquescentia = L_AUCTUS_ASCENDENS;
// fall through
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a compilation error on this comment

@eroux
Copy link
Contributor

eroux commented Dec 24, 2015

I got an aswer: the lines are indeed too think (for greciliae only), here is a comparison with other cavum glyphs:

cavum

I admit I'm a bit surprised as the thickness is the same, but the eye doesn't realize that apparently... do you think you could make lines a bit thicker?

@henryso
Copy link
Contributor Author

henryso commented Dec 24, 2015

I'm a bit confused. You say the lines are too thick, but then you ask that I make the lines thicker?

Edit: Never mind, I read "think" as "thick", and I guess you meant "thin".

@henryso
Copy link
Contributor Author

henryso commented Dec 24, 2015

Done. Let's not merge this until the person is happy with the shape.

@henryso
Copy link
Contributor Author

henryso commented Dec 24, 2015

Sample:

image

@rpspringuel
Copy link
Contributor

Looks better, but the upper right-hand corner of the oriscus cavum looks a bit thin.

@henryso
Copy link
Contributor Author

henryso commented Dec 24, 2015

Updated. How does this look?

image

@rpspringuel
Copy link
Contributor

To my eye that looks perfect.

@henryso
Copy link
Contributor Author

henryso commented Dec 24, 2015

Thanks for helping me to improve the shape. Let's see if @eroux's contact agrees.

@eroux
Copy link
Contributor

eroux commented Dec 25, 2015

Sorry, the words "thin" and "thick" are confusing for me! Anyway, this looks perfect, merging

eroux added a commit that referenced this pull request Dec 25, 2015
Added oriscus cavum glyphs.
@eroux eroux merged commit f2d7128 into gregorio-project:develop Dec 25, 2015
@eroux eroux mentioned this pull request Dec 25, 2015
@henryso henryso deleted the fix-724 branch December 25, 2015 14:24
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