-
Notifications
You must be signed in to change notification settings - Fork 499
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
Adjustable font text size #496
Comments
@giomfo: please can we try to get this into the upcoming release? it's starting to become quite a contentious point.... |
Related to issue element-hq#496. This patch is not exhaustive - it doesn't make everything use Dynamic Type fonts. Still, it makes most of the (frequently used) UI scalable, which should be a nice first step and an improvement for people with visibility problems. Even in the generic case (no visibility problems and font size increase set in Accessibility settings), with this patch, the fonts appear a little larger and more legible than before (addresses issue element-hq#1487). This patch defines some basic fonts, listens for a `UIContentSizeCategoryDidChangeNotification` and maintains these fonts updated accordingly. To let the rest of the app know about such font size changes, we send the same `kRiotDesignValuesDidChangeThemeNotification` which gets dispatched when people change between the Dark/Light themes. Since the intention is similar (UI settings update & redraw), it seemed natural to repurpose it. Not all fonts are assigned dynamically (`widget.font = ..`), so not all widgets use the new font values from `RiotDesignValues`. In other places, we simply configure the Dynamic Type font inside the `.xib` file. UI widgets which have their font defined in the `.xib` file do not get immediately updated. Using the "Automatically Adjust Font" property seems like an easy viable solution, but it requires iOS>=10, so it hasn't been done. Signed-off-by: Slavi Pantaleev <[email protected]>
Related to issue element-hq#496. This patch is not exhaustive - it doesn't make everything use Dynamic Type fonts. Still, it makes most of the (frequently used) UI scalable, which should be a nice first step and an improvement for people with visibility problems. Even in the generic case (no visibility problems and font size increase set in Accessibility settings), with this patch, the fonts appear a little larger and more legible than before (addresses issue element-hq#1487). This patch defines some basic fonts, listens for a `UIContentSizeCategoryDidChangeNotification` and maintains these fonts updated accordingly. To let the rest of the app know about such font size changes, we send the same `kRiotDesignValuesDidChangeThemeNotification` which gets dispatched when people change between the Dark/Light themes. Since the intention is similar (UI settings update & redraw), it seemed natural to repurpose it. Not all fonts are assigned dynamically (`widget.font = ..`), so not all widgets use the new font values from `RiotDesignValues`. In other places, we simply configure the Dynamic Type font inside the `.xib` file. UI widgets which have their font defined in the `.xib` file do not get immediately updated. Using the "Automatically Adjust Font" property seems like an easy viable solution, but it requires iOS>=10, so it hasn't been done. Signed-off-by: Slavi Pantaleev <[email protected]>
Related to issue element-hq#496. This patch is not exhaustive - it doesn't make everything use Dynamic Type fonts. Still, it makes most of the (frequently used) UI scalable, which should be a nice first step and an improvement for people with visibility problems. Even in the generic case (no visibility problems and font size increase set in Accessibility settings), with this patch, the fonts appear a little larger and more legible than before (addresses issue element-hq#1487). This patch defines some basic fonts, listens for a `UIContentSizeCategoryDidChangeNotification` and maintains these fonts updated accordingly. To let the rest of the app know about such font size changes, we send the same `kRiotDesignValuesDidChangeThemeNotification` which gets dispatched when people change between the Dark/Light themes. Since the intention is similar (UI settings update & redraw), it seemed natural to repurpose it. Not all fonts are assigned dynamically (`widget.font = ..`), so not all widgets use the new font values from `RiotDesignValues`. In other places, we simply configure the Dynamic Type font inside the `.xib` file. UI widgets which have their font defined in the `.xib` file do not get immediately updated. Using the "Automatically Adjust Font" property seems like an easy viable solution, but it requires iOS>=10, so it hasn't been done. Signed-off-by: Slavi Pantaleev <[email protected]>
I have iPhone 7 and yes, it’s hard to read the text for my tired eyes. |
This really needs to come in, number one complaint I have from people I try to introduce to Riot. It's way too small by default with no way of adjusting it. |
My husband has achromatopsia and my best friend also has an eye condition, they need to change the font size in order to see anything in the app. Not being able to do so is an accessibility nightmare. |
tiny font size hurts my eyes. decided to quickly search issues and found this.... nearly 3 years ago requested?! |
Second that. Adjustable font size is needed. |
Yes please - this is a REALLY good product, and i appreciate all the work that has gone into it! Adding in the larger font option would allow for better/more adoption! :-) |
For iOS users who need this feature (for accessibility reasons, etc.), I recommend trying FluffyChat. |
Today the systems settings regarding text size are mostly ignored - system text size does appear to be honoured in the contacts list (where it causes problems like #3386). |
The key issue is Accessibility settings:
Please, please, please. |
Seconding and following up on this... Looks like it's been an issue for quite some time. This continues to be the biggest complaint I get regarding usability of the application. |
I am amazed that this issue has been open for years with no fix in sight. PLEASE add a font size setting to the app -- it is a serious usability & accessibility issue. |
What @catwithbanana said. People with bad eyesight (me) wold appreciate that. |
I would agree. It seems like this would be a huge win and is a big ticket item |
Please this is old and really needed |
What would it take to resolve this? I can’t believe this has been an issue since 2018 and still has yet to be addressed. |
Really sorry to be rude and correct you this way. |
@Zafrola no worries! Even better - 2016! still sad though |
People keep asking me about this, it would be great if it happened one day. :/ |
Hey developers, is there a voting method we can use to convey the importance of this feature or rather requirement? Half your population have trouble reading the messages in ios. |
Given that it's been at least 6 years and the font size is still exactly the same, it's obvious that element-ios' developers think that whatever the font size is now is reasonably good. Even if Dynamic font sizes are difficult to implement (which they're not very, since I've done some initial work on it 5 years ago in #1562), they could have at least raised the default font size for everyone to a sane (larger) value. There are other clients (e.g. Fluffychat) that use a larger font size by default and that support increasing the font size even further. Your interests are better served elsewhere anyway. |
we were also in the need for font increase capability but since it has no progress we moved to fluffy chat that supports this option. |
Adding a reply to this issue with the current state of our support for Dynamic Type on iOS. Whilst we would very much like to support this in our existing app, the core screens (room lists and timeline) are very old now and require significant modernisation. We have chosen instead to focus our efforts on re-writing these as part of ElementX (our next gen iOS client that uses the Rust SDK). Whilst it will be some time before we have anything to release that is even Alpha, I would like to emphasise that Dynamic Type was one of the first things we considered in the prototyping stage and already have support for it in the both the room list and timeline: element-hq/element-x-ios#44 (comment) Sorry I don't have anything more immediate to share, but felt it was worth adding some perspective to this issue. |
What a shameless excuse. |
Feedback from a user received over email:
|
* Add DTCoreText dependency * Base implementation with DTCoreExtension and a dedicated package for Parser * Fix broken iOS tests * Set default paragraph style everywhere and improve code block display * Create a test target for HTMLParser and clean some code * Remove convenience NSAttributedString init and replace with `HTMLParser.parse` * Clean Swfitlint warning * Add HTMLParserStyle * Rename swizzling methods with a mx prefix * Remove some unused code and update documentation * Move snapshot tests to a dedicated test target for better visibility * Add lists snapshot tests * Clear Swiftlint warning and separate SnapshotTests in multiple test cases * Actually display multi-line quote * Fix a CSS issue and improve applying background style * Fix code block test for block display style * Avoid HTML parser owning a copy of the parser style * Store background style data in attributes & clean up some code * Clean and remove some unused code
Any update on this? Text sizes are really small |
We will not make it in Element-iOS. It is already available in ElementX, the new coming app. You can see it action here (this is the first animation I found. EX looks much nicer now). |
Update from 2024. Font size still cannot be changed. This caused half of my family to swap Element for Signal, because they are now over 70 years old and cannot read the font. I wonder if the developers just fugure that old people will die off at some point because they are not the developers' future so why create a product for them, but perhaps this is too cynical. I know ElementX was released, but everybody in my family and friends use iphone 6,7 and SE1 on ios 15.8.3 ( last release a few months ago), and they are not going to buy replacement iphones. Much cheaper to use Signal. Finally ElementX font sizes cannot be changes on ios 17 either: So it is a joke. What a wasted oppertunity. Good bye. |
Fluffy chat requires user registration for github, gmail GitLab,Facebook,Apple credentials which is odd. |
FluffyChat does not require people to use those services. How users can sign in is decided by the homeserver. FluffyChat apparently doesn't support in-app registration anymore, but if you can register using a different app (e.g., Element web), you can then sign in using FluffyChat. |
Hi There are barriers for everyday people who cannot, or want, to read small text to sign up to access messaging on Matrix servers. Element and ElemenX do not support font size changed but do allow registration, and Fluffy Chat does provide font size changes but not registration. A new user who requires font size changes has to: |
Vector should obey the system text size setting: Gary has his system text size turned up but it's still tiny in vector which makes it hard to read.
The text was updated successfully, but these errors were encountered: