Igalia / meta-webkit

Yocto / OpenEmbedded layer for WebKit based engines and browsers
MIT License
125 stars 69 forks source link

Prefer DejaVu instead of Bitstream Vera fonts #320

Closed aperezdc closed 2 years ago

aperezdc commented 2 years ago

Prefer the DejaVu fonts, which are derived from Vera and add better Unicode coverage, which means they are more suitable as fallback fonts.

psaavedra commented 2 years ago

should it be replace as well for GTK?

recipes-browser/wpewebkit/wpewebkit.inc:    ttf-bitstream-vera \
recipes-browser/webkitgtk/webkitgtk_2.34.1.bb:                     ttf-bitstream-vera \

and also in recipes-browser/packagegroups/packagegroup-wpewebkit-depends.bb: ttf-bitstream-vera

aperezdc commented 2 years ago

should it be replace as well for GTK?

recipes-browser/wpewebkit/wpewebkit.inc:    ttf-bitstream-vera \
recipes-browser/webkitgtk/webkitgtk_2.34.1.bb:                     ttf-bitstream-vera \

and also in recipes-browser/packagegroups/packagegroup-wpewebkit-depends.bb: ttf-bitstream-vera

Good point! I will do the change there as well and re-upload. Thanks for the suggestion.

psaavedra commented 2 years ago

The name of the ttf-dejavu packages is

ttf-dejavu-sans \
ttf-dejavu-sans-mono \
ttf-dejavu-sans-condensed \
ttf-dejavu-serif \
ttf-dejavu-serif-condensed \
ttf-dejavu-mathtexgyre \
ttf-dejavu-common"
aperezdc commented 2 years ago

The name of the ttf-dejavu packages is

ttf-dejavu-sans \
ttf-dejavu-sans-mono \
ttf-dejavu-sans-condensed \
ttf-dejavu-serif \
ttf-dejavu-serif-condensed \
ttf-dejavu-mathtexgyre \
ttf-dejavu-common"

You're right, ttf-dejavu is the name of the recipe, but it generates several packages. Somehow I always get confused by multi-package recipes 🤦🏼

For use as fallback fonts it should be enough to have sans, serif, and monospace variants. Technically CSS also specifies the cursive and fantasy generic font families, but in practice they are not used often, and the sans fallback font will be used if there is none more specific.