-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
The light-custom and dark-custom themes still use the now-replaced Nunito fonts? #15095
Comments
I believe it was intentional to not affect any existing themes. |
In my humble opinion, if the custom themes won’t be changed and adapted, then custom theming won’t improve. I don’t see any advantages here. I strongly believe that anyone who uses custom theming and has created their own custom themes is looking forward to improvement of custom theming. (Custom) Theming is nothing you do once and you’re done. Finished. Things will change, things will break, things will be improved. We shouldn’t keep legacy and outdated and buggy code just because someone might be using it and we’re afraid to make them adapt their themes and, in the end, deliver them a better experience and more possibilities and features. From another side of view, anyone who just starts using Element and wants to create their first custom theme, they will be presented the legacy theme and that’s just confusing. They probably want to change the default theme they’re seeing, but if changing some colors then automatically changes the overall appearance of the theme … that’s just unexpected and weird. |
The custom themes are specifically pinned to the legacy themes due to compatibility and time constraints. The idea is to port it over to the new themes eventually. |
As of now Element harcodes the default font in all custom themes: https://github.com/matrix-org/matrix-react-sdk/blob/develop/res/themes/light-custom/css/light-custom.scss#L3. This prevents adding custom fonts to custom themes. Another Element bug ([Issue #15095](element-hq/element-web#15095)) replaces the default font Inter with the previous default Nunito. We expect to add custom fonts once this is resolved.
This one took me quite some time to figure out … while the light and dark theme use
font-family: Inter, …
, the light-custom and dark-custom themes still use the now-replaced Nunito font …https://github.com/matrix-org/matrix-react-sdk/blob/develop/res/themes/light-custom/css/light-custom.scss#L3
https://github.com/matrix-org/matrix-react-sdk/blob/develop/res/themes/legacy-light/css/_fonts.scss#L17-L34
Can this please be updated as well? Thank you.
The text was updated successfully, but these errors were encountered: