On Thu, 16 Apr 2020 12:40:24 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> As per JavaFx 700 font weight is considered to be bold but webkit is using >> 600 font weight for text to become bold. to >> fix issue, use boldWeightValue() function which uses 700 font weight rather >> than isFontWeightBold() which compare >> against 600 font weight. > > Can you add a unit test to go along with this fix? Per the opentype spec, 700 is bold. 600 is semi-bold https://docs.microsoft.com/en-us/typography/opentype/spec/os2#usweightclass CSS agrees : https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight So are you saying webkit has been using bold at a lower weight than these specs suggest ? I see the logic all comes from Source/WebCore/platform/graphics/FontSelectionAlgorithm.h I suppose the existing code thinks that if we have reached what that file calls the bold threshold of 600 then we should use bold. It isn't necessarily "wrong" but I think I agree that it is more important to be consistent with the rest of Java FX ... which I believe is the point of this change ? ------------- PR: https://git.openjdk.java.net/jfx/pull/180
