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

U+03C6 and U+03D5 swapped (Forms of the Greek letter Phi) #120

Open
gandaro opened this issue Apr 10, 2016 · 7 comments
Open

U+03C6 and U+03D5 swapped (Forms of the Greek letter Phi) #120

gandaro opened this issue Apr 10, 2016 · 7 comments
Labels

Comments

@gandaro
Copy link

gandaro commented Apr 10, 2016

According to the Unicode Standard, U+03C6 should use the loopy form of the Greek letter Phi and U+03D5 should use the straight form of the letter. Source Code Pro uses the straight form for U+03C6 and the loopy form for U+03D5.

Excerpt from the relevant paragraphs in the Unicode Standard:

For mathematical and technical use, the straight form of the small phi is an important symbol and needs to be consistently distinguishable from the loopy form. […]

[…] fonts that also intend to support technical use of the Greek letters should use the loopy form [for U+03C6] to ensure appropriate contrast with the straight form used for U+03D5.

See Unicode 8.0.0, Chapter 7.2, “Representative Glyphs for Greek Phi”

@miguelsousa
Copy link
Member

This issue relates to issue #70 in the Source Sans repository.

The shape of U+03C6 can be the loop form or the full stroke form. The form used will depend on the genre of the design and on the designer's preferences.
On the other hand, the shape of U+03D5 should indeed be the straight form, no matter the style/genre of the typeface.

pauldhunt added a commit to pauldhunt/SourceCodePro that referenced this issue May 12, 2016
Regenerates Instances. Adds rules to <family.fea> and <italics.fea> to shift <breinverteddoublecmb> and <macrondoublebelowcmb> when they would otherwise collide with extenders. Cleans up feature files a bit. Fixes the following bugs:
adobe-fonts#115: COMBINING DOUBLE INVERTED BREVE (U+0361) causes characters to overlap
adobe-fonts#120: U+03C6 and U+03D5 swapped (Forms of the Greek letter Phi)
@gandaro
Copy link
Author

gandaro commented Jul 19, 2016

Thanks for your effort. It seems like U+03C6 and U+03D5 look the same now, though. Was that intended?

@pauldhunt
Copy link
Contributor

pauldhunt commented Jul 19, 2016 via email

@gandaro
Copy link
Author

gandaro commented Jul 19, 2016

U+03C6 to use the curly form, as Unicode suggests:

[…] fonts that also intend to support technical use of the Greek letters should use the loopy form [for U+03C6] to ensure appropriate contrast with the straight form used for U+03D5.

See Unicode 8.0.0, Chapter 7.2, “Representative Glyphs for Greek Phi”

@pauldhunt
Copy link
Contributor

I thought I had fixed this. If it is still an error, then I apologize deeply. I will take another crack on it on the next update. Thanks for bringing this to my attention. 🙇

@pauldhunt pauldhunt reopened this Jul 19, 2016
@moyogo
Copy link

moyogo commented Jul 19, 2016

No! U+03C6 is not the loopy form. U+03C6 can have whatever forms fits the style the type designer chooses, it just happens to be the loopy form in most fonts and therefore the reference glyph in the Unicode charts uses that. U+03D5 is a Math symbol and should have the straight form.

The full sentence in TUS 8.0 is

Fonts used primarily for Greek text may use either glyph form for U+03C6, but fonts that
also intend to support technical use of the Greek letters should use the loopy form to
ensure appropriate contrast with the straight form used for U+03D5.

@gandaro
Copy link
Author

gandaro commented Jul 19, 2016

Yes, it is optional, but Unicode recommends it for fonts that are intended to support technical use – as Source Code Pro certainly does. If you don’t like it though, you can leave it as it is now.

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

No branches or pull requests

4 participants