Closed davelab6 closed 8 years ago
One last thing:
ER: jsMath-cmbx10.ttf space -2 nbsp 0: Change space advanceWidth to 0
should be
ER: jsMath-cmbx10.ttf space -2 nbsp 0: Change nbsp advanceWidth to -2
https://github.com/google/fonts/blob/master/tools/space_check.py has been released which provides much more thorough checking of the nbsp glyph, but no autofixing.
There is a nasty bug in fontbakery-fix-nbsp.py where if the nbsp advanceWidth value is less than the space advanceWidth value, it recommends changing the value of space. This is wrong, nbsp should always match space, never the other way round.
This is okay, but it should say,
ER: DroidSansThai-Bold.ttf space 532 nbsp None: Add nbsp with advanceWidth 532
with advanceWidth
should be added to all recommendation stringsThis is wrong, should say,
ER: DroidSansThai-Bold.ttf space 532 nbsp 0: Change nbsp advanceWidth to 532
This is weird but okay.
This is all good
These last 3 are also wrong, but seems the same as the above checkmark point.