- Issue created by @Kristen Pol
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
💯 — I've seen this happen too, but somehow currently cannot reproduce this.
We shouldn't be loading a custom font. We should use a font stack that uses system fonts.
Per https://github.com/FortAwesome/Font-Awesome/issues/1124, it looks like this is a d.o terms of service violation too, given that the Inter font license (https://github.com/rsms/inter/blob/master/LICENSE.txt) appears to not be GPL-compatible.
@lauriii, @jessebaker and I discussed this a week ago, but apparently @jessebaker didn't file an issue for it: https://www.drupal.org/project/issues/experience_builder?text=font+stack... →
- 🇫🇮Finland lauriii Finland
It seems that Radix Themes is using system fonts by default: https://www.radix-ui.com/themes/docs/theme/typography#font-family. I wonder if we could just remove our overrides for the typography and it would work as expected?
- First commit to issue fork.
- Merge request !2173470552: Use system font stack rather than loading the custom Inter font → (Merged) created by omkar-pd
- Status changed to Needs review
3 months ago 10:20am 28 August 2024 - Assigned to hooroomoo
- Status changed to RTBC
3 months ago 11:18am 28 August 2024 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Thanks @omkar-pd!
This font was introduced by @hooroomoo in ✨ Create UI scaffolding for the primary panel (left sidebar) Needs work . Assigning to them for final review to ensure we're not overlooking something.
- First commit to issue fork.
-
hooroomoo →
committed 7cb3623d on 0.x authored by
omkar-pd →
Issue #3470552 by omkar-pd, Wim Leers, kristen pol, lauriii: Follow-up...
-
hooroomoo →
committed 7cb3623d on 0.x authored by
omkar-pd →
- Issue was unassigned.
- Status changed to Fixed
3 months ago 3:28pm 28 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.
