search

13rac1 / emojione-color-font

End of Life. Switch to https://github.com/eosrei/twemoji-color-font
Other
915 stars 46 forks source link

Breaks non-breakable spaces #63

Closed nclm closed 8 years ago

nclm commented 8 years ago

I had this bug on my Linux system for months but never investigated further. Then today I was playing with emoji fonts and fontconfig files and discovered it comes from here.

Non-breakable spaces appears too wide.

The wideness of this space looks weird and over time it is distracting when reading.

I think Bitstream Vera itself is fine. The emoji font doesn’t contain a non-breakable space character, but it does contain a simple space one, which may be the one that has larger metrics; and in a way or another   defaults back to it.

Using Firefox on Linux. Webkit/Blink browsers seem to display fine.

13rac1 commented 8 years ago

Huh. I actually use the exact space width of Deja Vu and Vera: https://github.com/eosrei/emojione-color-font/blob/b6d5340ce0bbfe6dc5676038550cb296da023501/scfbuild.yml#L13-L14

.ttx files created with ttx from fonttools.

unitsPeEm: DejaVuSans.ttx: <unitsPerEm value="2048"/> Vera.ttx: <unitsPerEm value="2048"/>

"space" width: DejaVuSans.ttx: <mtx name="space" width="651" lsb="0"/> Vera.ttx: <mtx name="space" width="651" lsb="0"/>

Hmm... problem is the "nbsp" width: DejaVuSans.ttx: <mtx name="nonbreakingspace" width="651" lsb="0"/> Vera.ttx: <mtx name="nonbreakingspace" width="1303" lsb="0"/>

Not sure what I can do about this... Could build a merged version like the Windows version??? I don't understand why Vera nbsp is so much bigger.

nclm commented 8 years ago

Good find. There’s probably some place to bug report for Vera. Thanks!

nclm commented 8 years ago

I sent an e-mail to fonts(@)gnome(.)org which appears to host Bitstream Vera.

I think this bug can be closed as it does not belong here.

Unless… can you set your own U+00A0 with a width of 651 in the EmojiOne font and make it priority in place of Vera’s one?

13rac1 commented 8 years ago

I could create the U+00A0, but since Vera is the config default it will override. Not much I can do...