Skip to content

Commit b86a44f

Browse files
committed
[arabic_high_hamza] Make the size of high hamza a warning
It seems to be debatable since high hamza is used also in Kazakh where small high hamza is acceptable. Instead of complicated and possibly error prune language detection, lets make it a warning.
1 parent 633d836 commit b86a44f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Lib/fontbakery/checks/arabic_high_hamza.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from fontbakery.prelude import FAIL, SKIP, Message, check
1+
from fontbakery.prelude import FAIL, SKIP, WARN, Message, check
22
from fontbakery.utils import get_glyph_name
33

44

@@ -70,9 +70,9 @@ def check_arabic_high_hamza(ttFont):
7070
high_hamza_area = area_pen.value
7171

7272
if abs((high_hamza_area - hamza_area) / hamza_area) > 0.1:
73-
yield FAIL, Message(
73+
yield WARN, Message(
7474
"glyph-area",
7575
"The arabic letter high hamza (U+0674) should have roughly"
76-
" the same size the arabic letter hamza (U+0621),"
77-
" but a different glyph outline area was detected.",
76+
" the same size the arabic letter hamza (U+0621) while raised"
77+
" above baseline, but a different glyph outline area was detected.",
7878
)

0 commit comments

Comments
 (0)