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

The light-custom and dark-custom themes still use the now-replaced Nunito fonts? #15095

Open
fooness opened this issue Sep 4, 2020 · 3 comments
Labels
A-Themes-Custom Custom theme variables or support A-Theming T-Defect

Comments

@fooness
Copy link
Contributor

fooness commented Sep 4, 2020

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.

@t3chguy
Copy link
Member

t3chguy commented Sep 4, 2020

I believe it was intentional to not affect any existing themes.

@fooness
Copy link
Contributor Author

fooness commented Sep 4, 2020

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.

@turt2live
Copy link
Member

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.

@jryans jryans removed the Z-UI/UX label Mar 8, 2021
dipyamanroy added a commit to dipyamanroy/element-themes that referenced this issue Oct 24, 2021
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.
@aaronraimist aaronraimist added A-Themes-Custom Custom theme variables or support A-Theming labels Oct 25, 2021
@t3chguy t3chguy removed the Z-Labs label Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Themes-Custom Custom theme variables or support A-Theming T-Defect
Projects
None yet
Development

No branches or pull requests

6 participants