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

Custom font size is limited between 13 and 20 #23068

Closed
Mikaela opened this issue Aug 13, 2022 · 7 comments
Closed

Custom font size is limited between 13 and 20 #23068

Mikaela opened this issue Aug 13, 2022 · 7 comments
Labels
A11y A-Appearance A-User-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product More input needed from the Product team

Comments

@Mikaela
Copy link
Contributor

Mikaela commented Aug 13, 2022

Steps to reproduce

  1. Settings, UI/Appearance, Font size
  2. The slider can go between 13 and 18
  3. I want a smaller font so I click custom font size
  4. I cannot go below 12 as it says that it must be between 12 and 18.

Outcome

What did you expect?

I expected to be able to specify as custom font size instead of being restricted to the slider above (which I thought allowed the same numbers, but apparently caps to 18 instead of 20).

What happened instead?

I am forced to stay between 13 and 20.

Operating system

"Fedora release 36 (Thirty Six)"

Browser information

Mozilla/5.0 (X11; Linux x86_64; rv:105.0) Gecko/20100101 Firefox/105.0 ID:20220812093714

URL for webapp

develop.element.io

Application version

Element-versio: 39eee10-react-8db7766a4012-js-3f6f5b69c7a1 Olm-versio: 3.2.12

Homeserver

pikaviestin.fi is on Synapse 1.64.0

Will you send logs?

No

@Mikaela
Copy link
Contributor Author

Mikaela commented Aug 13, 2022

My real problem is that left sidebar is using too big font for me to be able to utilise it.

The workaround to this is setting font size to 15 and set page zoom level to 80 % in the browser.

@SimonBrandner SimonBrandner added S-Minor Impairs non-critical functionality or suitable workarounds exist A-User-Settings A-Appearance A11y O-Occasional Affects or can be seen by some users regularly or most users rarely labels Aug 13, 2022
@andybalaam
Copy link
Contributor

A possible workaround is to use my (unsupported, experimental) tampermonkey script here: https://gitlab.com/andybalaam/element-web-tweaks/

@Diablo-D3
Copy link

Any update on this? Seems like a simple fix to remove the check entirely.

@andybalaam andybalaam added the X-Needs-Product More input needed from the Product team label Dec 14, 2022
@andybalaam
Copy link
Contributor

@vector-im/product can someone comment on this? We currently prevent the user from choosing the font size they want, with a message "Custom font size can only be between 13 pt and 20 pt".

I think a warning might be more appropriate - this is actively preventing some people making the app more accessible for their needs.

@Diablo-D3
Copy link

This bug intersects also with how Element considers text size; this sets the base value, everything else seems to be relative offsets of this.

#23913 basically illustrates the problem where font size in the prefs does not directly set the message body text size. So, a lot of stuff is kinda wrong here, and limiting the user from fixing it is kinda pear-shaped.

@daniellekirkwood
Copy link
Contributor

I believe this has been fixed...

These are the available font sizes today in Nightly
Screenshot 2024-02-26 at 18 38 16

@t3chguy
Copy link
Member

t3chguy commented Feb 26, 2024

Fixed by matrix-org/matrix-react-sdk#12246

@t3chguy t3chguy closed this as completed Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11y A-Appearance A-User-Settings O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Product More input needed from the Product team
Projects
None yet
Development

No branches or pull requests

6 participants