search

EbenSorkin / Merriweather4

This will be the new home of Merriweather when the build process is successful. Until then it is a testbed only.
SIL Open Font License 1.1
8 stars 2 forks source link

v4.003 Italic Interpolation issues with diacritics #10

Closed kenmcd closed 3 months ago

kenmcd commented 5 months ago

v4.003 Italic Interpolation issues with diacritics Some of the diacritics move off center in different masters. File: Merriweather-Italic[opsz,wdth,wght].ttf Glyphs: /uni04EA/uni04F1/uni0439.sc/uni045D.sc/uni045C.sc/AEacute/uni04EA/uni04F1/uni0439.sc/uni045D.sc/uni045C.sc

2024-07-02_13-49-34

Some of the other diacritics seem to move up and down when they probably should not. So you may want to review the height across masters.

kenmcd commented 5 months ago

and this /dieresiscombcy.case

2024-07-02_14-07-47

m4rc1e commented 5 months ago

Thanks @kenmcd. I always love receiving these reports.. I'm currently writing a tool to catch these anomalies.

m4rc1e commented 5 months ago

I have a prototype working nicely in Glyphsapp sources. Here are some of my findings:

Screenshot 2024-07-03 at 15 26 42 Screenshot 2024-07-03 at 15 27 58 Screenshot 2024-07-03 at 15 28 36

I should probably invest the time to get this working on TTFs.

m4rc1e commented 5 months ago

@EbenSorkin Please could you inspect the following glyphs:

semisoftsign-cy.sc 29428.0 iishorttail-cy 12817.0 tetse-cy 12576.0 Iebreve-cy 9174.0 Io-cy 9154.0 Iegrave-cy 9143.0 addressedtothesubject 1061.0 ddotaccent 708.0 Dzcaron 619.0 trademark 619.0 f_f_l.sc 612.0 dzcaron 573.0 Edieresis-cy 543.0 iishort-cy.sc 543.0 f_f_i.sc 537.0 udieresis-cy 535.0 aring 524.0 endlowtonemod 521.0 Rstroke 500.0 iigrave-cy.sc 492.0 adotbelow 486.0 nj.sc 474.0 dz 473.0 commercialMinusSign 467.0 uhungarumlaut-cy 461.0 zdotbelow 455.0 zcaron 443.0 whook 436.0 Obarreddieresis-cy 436.0 exclamdouble 433.0 emtail-cy.sc 423.0 Dz 420.0 ae.sc 420.0 aeacute.sc 420.0 aemacron.sc 420.0 aie-cy.sc 420.0 f_f.sc 411.0 fdotaccent 409.0 oe.sc 409.0 dz.sc 408.0 dzcaron.sc 408.0 uk-cy.sc 406.0 uk-cy 405.0 Ezhcaron 403.0 Kaverticalstroke-cy 403.0 enge-cy 398.0 komitje-cy 398.0 dittomark 397.0 tripleprime 394.0 longbar_part. 394.0 odotbelow 392.0 fl.sc 392.0 Uk-cy 391.0 numero 389.0 m.sc 387.0 macute.sc 387.0 mdotaccent.sc 387.0 mdotbelow.sc 387.0 em-cy.sc 387.0 radical 386.0 Emtail-cy 385.0 Nj 383.0 OE 382.0 yusbig-cy 382.0 omegaBroad-cy.sc 380.0 Dhook 378.0 NJ 378.0 tildeoverlaycomb 376.0 nje-cy 374.0 M 372.0 Macute 372.0 Mdotaccent 372.0 Mdotbelow 372.0 Em-cy 372.0 umacron-cy 372.0 Mhook 370.0 ij_acutecomb 370.0 dram-arm 369.0 ruble 369.0 naira 366.0 Fita-cy 365.0 nj 364.0 mhook.sc 362.0 Yusbig-cy 362.0 adblgrave.sc 361.0 Bhook 360.0 Nje-cy 360.0 IJ_acutecomb 357.0 IJ.loclNLD 357.0

There will be false positives in there but all these glyphs were flagged by my tool.

EbenSorkin commented 5 months ago

Yes. Thank you!

On Fri, Jul 5, 2024 at 5:32 AM Marc Foley @.***> wrote:

@EbenSorkin https://github.com/EbenSorkin Please could you inspect the following glyphs:

semisoftsign-cy.sc 29428.0 iishorttail-cy 12817.0 tetse-cy 12576.0 Iebreve-cy 9174.0 Io-cy 9154.0 Iegrave-cy 9143.0 addressedtothesubject 1061.0 ddotaccent 708.0 Dzcaron 619.0 trademark 619.0 f_f_l.sc 612.0 dzcaron 573.0 Edieresis-cy 543.0 iishort-cy.sc 543.0 f_f_i.sc 537.0 udieresis-cy 535.0 aring 524.0 endlowtonemod 521.0 Rstroke 500.0 iigrave-cy.sc 492.0 adotbelow 486.0 nj.sc 474.0 dz 473.0 commercialMinusSign 467.0 uhungarumlaut-cy 461.0 zdotbelow 455.0 zcaron 443.0 whook 436.0 Obarreddieresis-cy 436.0 exclamdouble 433.0 emtail-cy.sc 423.0 Dz 420.0 ae.sc 420.0 aeacute.sc 420.0 aemacron.sc 420.0 aie-cy.sc 420.0 f_f.sc 411.0 fdotaccent 409.0 oe.sc 409.0 dz.sc 408.0 dzcaron.sc 408.0 uk-cy.sc 406.0 uk-cy 405.0 Ezhcaron 403.0 Kaverticalstroke-cy 403.0 enge-cy 398.0 komitje-cy 398.0 dittomark 397.0 tripleprime 394.0 longbar_part. 394.0 odotbelow 392.0 fl.sc 392.0 Uk-cy 391.0 numero 389.0 m.sc 387.0 macute.sc 387.0 mdotaccent.sc 387.0 mdotbelow.sc 387.0 em-cy.sc 387.0 radical 386.0 Emtail-cy 385.0 Nj 383.0 OE 382.0 yusbig-cy 382.0 omegaBroad-cy.sc 380.0 Dhook 378.0 NJ 378.0 tildeoverlaycomb 376.0 nje-cy 374.0 M 372.0 Macute 372.0 Mdotaccent 372.0 Mdotbelow 372.0 Em-cy 372.0 umacron-cy 372.0 Mhook 370.0 ij_acutecomb 370.0 dram-arm 369.0 ruble 369.0 naira 366.0 Fita-cy 365.0 nj 364.0 mhook.sc 362.0 Yusbig-cy 362.0 adblgrave.sc 361.0 Bhook 360.0 Nje-cy 360.0 IJ_acutecomb 357.0 IJ.loclNLD 357.0

There will be false positives in there but all these glyphs were flagged by my tool.

— Reply to this email directly, view it on GitHub https://github.com/EbenSorkin/Merriweather4/issues/10#issuecomment-2210539007, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXP6E4PIGTLFMVSR3H3ZKZR33AVCNFSM6AAAAABKIJDZZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJQGUZTSMBQG4 . You are receiving this because you were mentioned.Message ID: @.***>

kenmcd commented 3 months ago

v4.006.(2024-08-20) - Merriweather-Italic[opsz,wdth,wght].ttf Only checked the few above I had checked previously. Still some issues

/uni04EA/uni045D.sc/uni045C.sc/AEacute/uni04EA/uni045D.sc/uni045C.sc/dieresiscombcy.case

2024-08-20_15-07-03

/dieresiscombcy.case has a width in 9 of 12 masters.

2024-08-20_15-12-40

EbenSorkin commented 3 months ago

Thanks!

I'll check if the latest version Marc is working on has these issues still and correct it if it does.

On Tue, Aug 20, 2024 at 6:20 PM kenmcd @.***> wrote:

v4.006.(2024-08-20) - Merriweather-Italic[opsz,wdth,wght].ttf Only checked the few above I had checked previously. Still some issues

/uni04EA/uni045D.sc/uni045C.sc/AEacute/uni04EA/uni045D.sc/uni045C.sc/dieresiscombcy.case

2024-08-20_15-07-03.gif (view on web) https://github.com/user-attachments/assets/9b72c652-5c43-4b12-8668-8ccbbecd6789

/dieresiscombcy.case has a width in 9 of 12 masters.

2024-08-20_15-12-40.gif (view on web) https://github.com/user-attachments/assets/c6e2b147-63fd-4bcd-bfee-dc0fb2e5fa9e

— Reply to this email directly, view it on GitHub https://github.com/EbenSorkin/Merriweather4/issues/10#issuecomment-2299858638, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXKX3SWQQGX56PTSQS3ZSO6MJAVCNFSM6AAAAABKIJDZZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJZHA2TQNRTHA . You are receiving this because you were mentioned.Message ID: @.***>

m4rc1e commented 3 months ago

@EbenSorkin I'm not working on these so please fix them. Thanks once again @kenmcd!

EbenSorkin commented 3 months ago

Ok! The fixes are made and pushed.

On Wed, Aug 21, 2024 at 3:54 AM Marc Foley @.***> wrote:

@EbenSorkin https://github.com/EbenSorkin I'm not working on these so please fix them. Thanks once again @kenmcd https://github.com/kenmcd!

— Reply to this email directly, view it on GitHub https://github.com/EbenSorkin/Merriweather4/issues/10#issuecomment-2301387766, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXP6GMWEIBX5RZITL5LZSRBUVAVCNFSM6AAAAABKIJDZZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRGM4DONZWGY . You are receiving this because you were mentioned.Message ID: @.***>