google / fonts

Font files available from Google Fonts, and a public issue tracker for all things Google Fonts
https://fonts.google.com
18.23k stars 2.62k forks source link

Update Montserrat with SSA glyphset #6923

Closed davelab6 closed 1 week ago

davelab6 commented 1 year ago

This is coming soon, WIP @ https://github.com/JulietaUla/Montserrat/commits/merge-forked-changes

also, check https://github.com/google/fonts/issues/4250

RosaWagner commented 1 year ago

@davelab6 "Google Fonts" for tracking issues "Traffic Jam" for tracking PR

davelab6 commented 12 months ago

@RosaWagner haha thanks :)

@juandelperal please update here when the project is ready - I'm guessing this/next week

EbenSorkin commented 11 months ago

We need to do a SCCMP test on the VF or the static instances as well

juandelperal commented 11 months ago

Good morning everyone,

https://github.com/JulietaUla/Montserrat/commits/merge-forked-changes is ready for review. Please let us know any improvements to be made before merging to master

davelab6 commented 11 months ago

@chrissimpkins this is a Q1 2 star :)

chrissimpkins commented 10 months ago

@chrissimpkins this is a Q1 2 star :)

Updated on the project board. 👍

chrissimpkins commented 7 months ago

Requires new manual TT hints

EbenSorkin commented 7 months ago

Woukd we want to complete any additions to the glyph set first? The newest African standard is I think slightly bigger than what they made. Maybe the order doesn’t matter?

On Mon, Mar 25, 2024 at 5:36 PM Chris Simpkins @.***> wrote:

Requires new manual TT hints

— Reply to this email directly, view it on GitHub https://github.com/google/fonts/issues/6923#issuecomment-2018960308, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXNARPQCYPHWM7G376TY2CKEDAVCNFSM6AAAAAA6WQCSG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJYHE3DAMZQHA . You are receiving this because you commented.Message ID: @.***>

juandelperal commented 7 months ago

Hi everybody!

Thanks @EbenSorkin for noticing this, where can we find the new SSA character set?

EbenSorkin commented 7 months ago

It is here: https://github.com/googlefonts/glyphsets/blob/main/data/results/txt/nice-names/GF_Latin_African.txt

EbenSorkin commented 7 months ago

It has been renamed to GF African and it is here:

https://github.com/googlefonts/glyphsets/blob/main/data/results/txt/nice-names/GF_Latin_African.txt

Also Beyond is here

https://github.com/googlefonts/glyphsets/blob/main/data/results/txt/nice-names/GF_Latin_Beyond.txt

It looks like the difference is now 4 more for African and 3 more for Beyond.

This text can be used to make a filter in Glyphs that shows what the additions are

-e.

On Tue, Mar 26, 2024 at 7:09 AM Juan del Peral @.***> wrote:

Hi everybody!

Thanks @EbenSorkin https://github.com/EbenSorkin for noticing this, where can we find the new SSA character set?

— Reply to this email directly, view it on GitHub https://github.com/google/fonts/issues/6923#issuecomment-2020140020, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXJ4ZRXIG56OCF4RCNDY2FCLZAVCNFSM6AAAAAA6WQCSG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE2DAMBSGA . You are receiving this because you were mentioned.Message ID: @.***>

EbenSorkin commented 7 months ago

Here:

https://github.com/JulietaUla/Montserrat/tree/merge-forked-changes

in the file "Montserrat.glyphs"

I see only these missing for GF African :

<GSGlyphInfo 0x600001422f80> oOpenmod, U:1D53, script:-, cat:Letter, subCat:Modifier case:noCase prod:uni1D53 direction:LTR sort:- comp:oopen > <GSGlyphInfo 0x60000947f7a0> A.circled, U:24B6, script:-, cat:Symbol, subCat:- case:noCase prod:uni24B6 direction:LTR sort:Letter.circled.001 comp:A > <GSGlyphInfo 0x600009478820> a.circled, U:24D0, script:-, cat:Symbol, subCat:- case:noCase prod:uni24D0 direction:LTR sort:Letter.circled.027 comp:a > <GSGlyphInfo 0x60003d355180> uniA7C7, U:A7C7, script:-, cat:Other, subCat:- case:noCase prod:uniA7C7 direction:LTR sort:uni0A7C7 comp:-> <GSGlyphInfo 0x60003d355220> uniA7C8, U:A7C8, script:-, cat:Other, subCat:- case:noCase prod:uniA7C8 direction:LTR sort:uni0A7C8 comp:->

Also to complete Beyond it is only :

<GSGlyphInfo 0x60000149cd20> lambda, U:03BB, script:greek, cat:Letter, subCat:- case:lower prod:- direction:LTR sort:gr034 comp:-> <GSGlyphInfo 0x60000149d4a0> chi, U:03C7, script:greek, cat:Letter, subCat:- case:lower prod:- direction:LTR sort:gr046 comp:-> <GSGlyphInfo 0x6000094b6ee0> YturnedSansSerif, U:2144, script:-, cat:Symbol, subCat:Math case:noCase prod:uni2144 direction:LTR sort:- comp:->

On Wed, Mar 27, 2024 at 11:13 AM Eben Sorkin @.***> wrote:

It has been renamed to GF African and it is here:

https://github.com/googlefonts/glyphsets/blob/main/data/results/txt/nice-names/GF_Latin_African.txt

Also Beyond is here

https://github.com/googlefonts/glyphsets/blob/main/data/results/txt/nice-names/GF_Latin_Beyond.txt

It looks like the difference is now 4 more for African and 3 more for Beyond.

This text can be used to make a filter in Glyphs that shows what the additions are

-e.

On Tue, Mar 26, 2024 at 7:09 AM Juan del Peral @.***> wrote:

Hi everybody!

Thanks @EbenSorkin https://github.com/EbenSorkin for noticing this, where can we find the new SSA character set?

— Reply to this email directly, view it on GitHub https://github.com/google/fonts/issues/6923#issuecomment-2020140020, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAQUQXJ4ZRXIG56OCF4RCNDY2FCLZAVCNFSM6AAAAAA6WQCSG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGE2DAMBSGA . You are receiving this because you were mentioned.Message ID: @.***>

emmamarichal commented 7 months ago

@chrissimpkins STATUS: We are waiting for the glyphset update + hinting. Maybe we should put it aside since we are still waiting for confirmations?

davelab6 commented 7 months ago

Thanks Eben! This will be done before the hinting starts June 1

chrissimpkins commented 7 months ago

Moving this back to Q3

m4rc1e commented 1 month ago

Should have this finished by the end of the quarter. Currently checking the hint transfer I made https://github.com/google/fonts/pull/8175

chrissimpkins commented 1 month ago

Updating to P2

cc @davelab6