steinbergmedia / bravura

Bravura music font, reference font for SMuFL (Standard Music Font Layout)
SIL Open Font License 1.1
308 stars 38 forks source link

U+E30E, accSagittal35LargeDiesisUp is upside down #75

Open Tychology opened 2 years ago

Tychology commented 2 years ago

U+E30E, accSagittal35LargeDiesisUp is upside down and looks exacly like U+E30F, accSagittal35LargeDiesisDown (As demonstrated in the current SMuFL specification):

grafik

It is supposed to look like this (from an older SMuFL PDF):

grafik

This issue seems to be introduced in https://github.com/steinbergmedia/bravura/commit/301087ca0b0d30b65d81bc3e718ff64b613e2a9a. In Bravura.svg, the path data for both symbols is identical:

    <glyph glyph-name="uniE30E" unicode="&#xe30e;" horiz-adv-x="375" 
d="M0 -47v219h31v-125c0 -84 31 -125 94 -125h31v437h32v-399l187 230v-112l-203 -250h-78c-63 0 -94 42 -94 125z" />
    <glyph glyph-name="uniE30F" unicode="&#xe30f;" horiz-adv-x="375" 
d="M0 -47v219h31v-125c0 -84 31 -125 94 -125h31v437h31v-399l188 230v-112l-203 -250h-78c-63 0 -94 42 -94 125z" />
dkeenan7 commented 1 year ago

I confirm that this glyph is wrong in both Bravura and Bravura Text, and hence the wrong reference glyph is being displayed on https://w3c.github.io/smufl/latest/tables/spartan-sagittal-single-shaft-accidentals.html. The correct glyph for U+E30E can be obtained from the file BravuraSagittalUpdate_v12.zip as attached to https://github.com/steinbergmedia/bravura/issues/53. Thanks @Tychology.