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

Greek review #21

Open
vv-monsalve opened this issue Sep 23, 2024 · 4 comments
Open

Greek review #21

vv-monsalve opened this issue Sep 23, 2024 · 4 comments

Comments

@vv-monsalve
Copy link
Collaborator

Hi, @josescaglione, I'm bringing here the feedback provided by @irenevlachou on Greek.

Her overall comment was:

There are a few spacing and kerning issues, and I’ve marked them in the Regular style and a few, the most extreme ones, in Extra Bold. The same issues occur in all styles, but I thought it would be a waste of time to repeat my comments and produce all these screenshots.

PPS-GRK-feedback-Irene.pdf

These PDFs correspond to the Round 1 Greek proofs produced at commit 5fc8902 Update kerning for Ramshorn.

Some reported issues might have already been solved since you continued working on the fonts.

@vv-monsalve
Copy link
Collaborator Author

vv-monsalve commented Oct 1, 2024

Hi, @josescaglione. I'm bringing here the feedback provided by @gerryleonidas on Greek.

PPS-Greek-summary comments-Gerry.pdf

These PDFs correspond to the Round 1 Greek proofs produced at commit 5fc8902 Update kerning for Ramshorn

@vv-monsalve
Copy link
Collaborator Author

Hi @josescaglione, I'm bringing here the feedback provided by @irenevlachou on Greek.

PPS-GRK-Irene-feedback-2.pdf

This PDF corresponds to the Round 2 Greek proofs produced at the commit 7e270f0 Update fea

@josescaglione
Copy link
Collaborator

Thanks for the revision Irene, we have gone over the documents again. I am leaving notes about the most relevant issues that were raised:

  1. Double entry in menu. We will keep the note and review after applying Builder2

  2. Various kerning and spacing notes in both documents. We addressed a number of issues raised by Irene and Gerry. However, please note that kerning is performed as class kerning where classes contain up to 7 variations in their design. The inconsistencies produced are intentional as they reflect the nature of organic and real handwriting.

  3. Tonos on upper cases. These were removed from the shuffler in order to be able to kern them correctly. Please note there is still a known issue with quotes+tonosCAP, similar to a problem found in Devanagari. This clash still happens and we are looking for a way to fix it.

  4. Anotelia. We decided to make the position consistent with questionmark

  5. iotatonos and iotadieresistonos. We decided to add alts for these two glyphs to include them in the shuffler and be able to avoid the clashes mentioned in the round2 document. This should be ready by tomorrow

@vv-monsalve
Copy link
Collaborator Author

Hi @josescaglione, I'm bringing here the feedback provided by @gerryleonidas on Greek.
This PDF corresponds to the Round 2 Greek proofs produced at the commit 7e270f0 Update fea.

PPS-Greek-Gerry-comments-2.pdf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants