Skip to content

fix: conversation and users names cut with large font (WPB-15168)#5002

Open
sbakhtiarov wants to merge 1 commit into
developfrom
feat/accessibility-large-fonts-titles
Open

fix: conversation and users names cut with large font (WPB-15168)#5002
sbakhtiarov wants to merge 1 commit into
developfrom
feat/accessibility-large-fonts-titles

Conversation

@sbakhtiarov

@sbakhtiarov sbakhtiarov commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-15168

https://wearezeta.atlassian.net/browse/WPB-15168

What's new in this PR?

Issues

Conversation and user names are not fully displayed when large font is set in accessibility settings.

Solutions

Use more lines for conversation and user names when font scaling is appied.

@sonarqubecloud

Copy link
Copy Markdown

Comment on lines +25 to +31
@Composable
@ReadOnlyComposable
fun maxTitleLines(): Int = if (LocalDensity.current.fontScale > DEFAULT_FONT_SCALE) {
EXPANDED_FONT_SCALE_MAX_LINES
} else {
DEFAULT_FONT_SCALE_MAX_LINES
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we have it either 1 or 4 or maybe add a bit more scaling?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure what you mean by "a bit more scaling". This is for maxLines value, so that the full title is visible, but it will not take full 4 lines unless it is needed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i mean depending on the font scale and how big it is
if there is a small difference between DEFAULT_FONT_SCALE and device then keep 1
a bigger difference use 2
and a large difference use 4

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if it is worth it, I really wanted to keep it simple

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants