RedHatOfficial / Overpass

Overpass open source web font family — Sponsored by Red Hat
http://overpassfont.org
SIL Open Font License 1.1
2k stars 92 forks source link

Overpass Mono is not recognized correctly #41

Open mcepl opened 7 years ago

mcepl commented 7 years ago

When I set Overpass Mono as a system mono font, Evolution gets confused and uses proportional font instead. With DejavuSans Mono everything works fine. Not sure what's the problem (and the pyramid of Evolution and libraries it uses is too high to be able to find out where the things break), but I thought it might be fruitful to compare metadata of these two fonts to find out whether Overpass Mono doesn't miss anything.

davelab6 commented 7 years ago

Fontbakery should catch all the mono metadata, eg Panose monospace bit and isFixedWidth bit

Cc @m4rc1e

On Dec 5, 2016 7:27 AM, "Matěj Cepl" notifications@github.com wrote:

When I set Overpass Mono as a system mono font, Evolution gets confused and uses proportional font instead. With DejavuSans Mono everything works fine. Not sure what's the problem (and the pyramid of Evolution and libraries it uses is too high to be able to find out where the things break), but I thought it might be fruitful to compare metadata of these two fonts to find out whether Overpass Mono doesn't miss anything.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/RedHatBrand/Overpass/issues/41, or mute the thread https://github.com/notifications/unsubscribe-auth/AAP9y21ugRC8f_A9wZCGmBYejnO8SZXOks5rFAMRgaJpZM4LEKvW .

m4rc1e commented 7 years ago

@davelab6 FB doesn't report any errors with isFixedWidth. However, Overpass Mono is panoseless. Perhaps setting a panose for each weight could solve this issue.

davelab6 commented 7 years ago

Then fb is broken because the monospace bit of panose must be set for monospace fonts!

Cc @felipesanches

On Dec 5, 2016 8:44 AM, "Marc Foley" notifications@github.com wrote:

@davelab6 https://github.com/davelab6 FB doesn't report any errors with isFixedWidth. However, Overpass Mono is panoseless. Perhaps setting a panose for each weight could solve this issue.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/RedHatBrand/Overpass/issues/41#issuecomment-264857064, or mute the thread https://github.com/notifications/unsubscribe-auth/AAP9yxnKvBhVtP5sOBP-691GFnVZne3Vks5rFBUhgaJpZM4LEKvW .

felipesanches commented 7 years ago

@davelab6 Would you like to make PANOSE mandatory ? Meaning that if a font file lacks a panose value, fontbakery would generate an error regarding that?

davelab6 commented 7 years ago

It's mandatory only for monospace fonts, others should be blank, or, if any file in a family has a value set, the value should be set to something in the other styles (although the value each is set to can vary)

On Dec 5, 2016 6:39 PM, "Felipe Corrêa da Silva Sanches" < notifications@github.com> wrote:

@davelab6 https://github.com/davelab6 Would you like to make PANOSE mandatory ? Meaning that if a font file lacks a panose value, fontbakery would generate an error regarding that?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/RedHatBrand/Overpass/issues/41#issuecomment-265014031, or mute the thread https://github.com/notifications/unsubscribe-auth/AAP9y_3st4Srjn-Vada0HBkcpfGa4KXKks5rFKCagaJpZM4LEKvW .

DelveFonts commented 7 years ago

Thanks for bringing this to our attention. We will get it fixed ASAP.

felipesanches commented 7 years ago

An I'll add an explicit test to make fontbakery emit an error if that problem is ever found again in a font file :-)

michaelkuhn commented 6 years ago

This is still happening for me with Overpass 3.0.2 on Fedora 28. Moreover, this does not seem to be limited to the monospace font: If I set Overpass as the system UI font and Overpass Mono as the system monospace font, Evolution uses some other proportional font for both instead. However, this only applies to the WebKit parts of Evolution (composer, e-mail view etc.) and not the GTK parts (folder and e-mail list etc.).