bBoxType / FiraSans

Fira is a large Open Font typeface family licensed under OFL
http://scripts.sil.org/OFL
Other
445 stars 36 forks source link

U+0305 Combining Overline doesn't render correctly. #27

Open atheistray opened 6 years ago

atheistray commented 6 years ago

I have to thank you for adding this glyph first. It was I who requested it in the first place. Unfortunately, I still can't get them to render properly on my phone. Here's a picture showing what I see: screenshot_2018-03-23-01-21-07-90 When using the typeface lato, the overline does show up but it's incorrectly shifted to the right. screenshot_2018-03-23-01-33-29-04 Here's what it should look like when rendered properly (still too high but better) using the typeface segoe UI: screenshot_2018-03-23-01-19-34-21 This is the website in the picture Thanks again for your great effort. ❤

carrois commented 6 years ago

Hi Ray.

Thanks for reporting. I can move the top anchor of /uni0305 so that it places between the two /o (see screenshot). I could also lengthen the stroke so that it fits perfectly over the two /o. But what if it is placed on top of more narrow glyphs? (see illustration with an example /idotless)? Not really sure here, do you have detailed knowledge on that?

bildschirmfoto 2018-03-23 um 10 50 37

I’m also wondering that the ' in "voo'doo" is not displayed with Fira. Is it /uni0027 or another glyph that might be missing?

atheistray commented 6 years ago

I've found out what the problem is. It's not really U+0305 Combining Overline that's missing. That glyph is actually U+035E Combining Double Macron, which is covered by both lato and segoe but not fira. Sorry, I didn't know I was looking for the wrong glyph. My apologies. The ʹ is U+2032 Prime.