search

aliftype / xits

XITS - OpenType implementation of STIX fonts with math support
SIL Open Font License 1.1
351 stars 35 forks source link

RadicalRuleThickness #4

Closed uvieth closed 14 years ago

uvieth commented 14 years ago

Hi Khaled,

I had a closer look at the OT math parameters and have one question: Why is RadicalRuleThickness (66) bigger than Overbar/UnderbarRuleThickness (50)? Changing RadicalRuleThickness would also make it consistent with RadicalExtraAscender.

I would have expected that all rule thicknesses were suppsed to be consistent.

Regards, Ulrik

khaledhosny commented 14 years ago

66 is the thickness of the horizontal bar of the top component of the composed radical (uni221A.top glyph), and I didn't see a good reason to change other rule thicknesses.

uvieth commented 14 years ago

OK, that's a good reason. I only looked at the numbers and didn't check the glyphs.

However that means, that the rule thickness in the design of the radical glyph is inconsistent with the overall rule thickness. Could be regarded as a STIX bug.

khaledhosny commented 14 years ago

OK, adding to the list of issues that I need to report back to STIX project.

khaledhosny commented 14 years ago

Just for the record, I found that 66 should be the default rule thickness not 50 that I was using (66 is consistently the thievishness of the hyphen, the minus, the strokes of the plus etc.) So in 56abbd9e6ae45847e2f9830358e8f193cbed887c I changes rule thickness to 66 and re-based all constants affected by it on the new value (following the recommendations of of OpenType math spec).