fonttools / fontbakery

🧁 A font quality assurance tool for everyone
https://fontbakery.readthedocs.io
Apache License 2.0
534 stars 99 forks source link

Re-enabled legacy_accents check #4705

Closed yanone closed 1 month ago

yanone commented 1 month ago

Description

Re-enabled check but dropped complaints about legacy accents as part of glyph compositions Relates to issue #4567

The two messages legacy-accents-width and legacy-accents-gdef are critical and should not have been disabled in my opinion.

Since the check was disabled rather than outright deleted, I interpret from this that back then there was no time to look into the matter in detail, and so I took the initiative to completely remove the legacy-accents-component WARN part but otherwise re-enabled the check as it was to still yield FAIL on the two other messages.

Checklist