Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Montserrat] Support for tabular number spacing limited to 700 weight #2553

Closed
lachieh opened this issue Jul 17, 2020 · 4 comments
Closed

[Montserrat] Support for tabular number spacing limited to 700 weight #2553

lachieh opened this issue Jul 17, 2020 · 4 comments

Comments

@lachieh
Copy link

lachieh commented Jul 17, 2020

Tabular spacing seems to be limited to the 700 weight:

Here is the 500 weight:
Screen Shot 2020-07-16 at 8 11 38 PM

Here is the 700 weight:
Screen Shot 2020-07-16 at 8 11 31 PM

Notice that the widths of each number are inconsistent (i.e. not good for tabular data). On the 700 example, the percentage signs are all aligned, 500 they are not. I have not looked at any of the other OpenType features to check if this is just tabular data or not.

Live test case:
https://codepen.io/lachieh/pen/yLeRqEw/left/

@thlinard
Copy link
Contributor

Hi,

All the boxes have aligned figures in my browser.

@lachieh
Copy link
Author

lachieh commented Jul 20, 2020

Thanks, I've updated the code to use the latest CSS import from fonts.google.com. I did have a local version of Montserrat bold installed, so I've removed it and now none of the boxes have aligned figures. I have confirmed this in latest Chrome and Firefox on macOS.

This is the import I have in the example now:

@import url('https://fonts.googleapis.com/css2?family=Montserrat:wght@100;200;300;400;500;600;700;800;900&display=swap');

And these are the URLs that are loading:

https://fonts.gstatic.com/s/montserrat/v14/JTUQjIg1_i6t8kCHKm45_QpRyS7m0dR9pA.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_aZA3gnD_vx3rCs.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_cJD3gnD_vx3rCs.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTUSjIg1_i6t8kCHKm459WlhyyTh89Y.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_ZpC3gnD_vx3rCs.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_bZF3gnD_vx3rCs.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_c5H3gnD_vx3rCs.woff2
https://fonts.gstatic.com/s/montserrat/v14/JTURjIg1_i6t8kCHKm45_epG3gnD_vx3rCs.woff2

What are some next steps I can take to resolve this?

@thlinard
Copy link
Contributor

Oh, yes. I believe Dave's answer is still valid:
#1582

Btw, the WOFF and WOFF2 files are already available In the latest version: https://github.com/JulietaUla/Montserrat/tree/master/fonts/webfonts

@lachieh
Copy link
Author

lachieh commented Jul 20, 2020

Ok, thanks. It would be great to add support for enabling OT features with a query string or something of the like. I understand it's not something that everyone uses, but it'd nice to be able to enable them selectively.

I'll close this one in favor of that other ticket or adding a feature request.

@lachieh lachieh closed this as completed Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants