system-ui in --font-sans font stack

[daniel-j-h]

Hey folks, I'm just seeing that we have system-ui in the sans font variable

pollen/src/modules/typography.ts

Line 53 in dc34314

sans: 'system-ui, -apple-system, Segoe UI, Roboto, Noto Sans, Ubuntu, Cantarell, Helvetica Neue',

and I was wondering if we should remove it as a default, since it's problematic for some users

• https://infinnie.github.io/blog/2017/systemui.html
• https://news.ycombinator.com/item?id=31543054

At the moment I work around it by overwriting the default in my project via

:root { --font-sans: .. }

Also wondering how we came up with the font stack, and if we can rely on something like https://systemfontstack.com?

[madeleineostoja]

I was under the impression that system-ui had significantly stabilised since those concerns? If that’s not the case then yep we can strip it out. Most production projects will overwrite these particular defaults anyway.

The current font stacks were taken from a postcss shim for system-ui, but your source looks v solid as well. Next time there’s a breaking change on the cards it would be worth updating them regardless.

As an aside it’ll be cleaner to overwrite / change anything you need in pollen with a config file rather than clobbering output css variables :)

[daniel-j-h]

From what I can see, system-ui still has its problems, but I also understand that this change would require a new pollen release.

Thanks for the config pointers, looks awesome to customize pollen so easily with a config file! I'll look into it! 🙌