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

ItemList - fix text_overrun_behavior #101899

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

havi05
Copy link
Contributor

@havi05 havi05 commented Jan 22, 2025

Fixes #100810 and fixes #74486
This pr also fixes the bug that item texts aren't rendered in rtl layout direction. (wasn't reported yet)
I removed an unnecessary addition and subtraction (line 1604 and line 1629).

I have changed the text offset when using icon_mode TOP so that the text is off center, but remains visible when using a small width.

Videos

icon_mode = Top:

2025-01-21.23-01-25.mp4

icon_mode = Left:

2025-01-22.09-31-47.mp4

@havi05 havi05 requested a review from a team as a code owner January 22, 2025 07:26
@bruvzg bruvzg self-requested a review January 22, 2025 07:30
@havi05 havi05 force-pushed the itemlist-fix-textoverrun branch 3 times, most recently from f9b47e4 to 287de4b Compare January 22, 2025 08:37
@AThousandShips AThousandShips changed the title Itemlist - fix text_overrun_behavior ItemList - fix text_overrun_behavior Jan 22, 2025
@AThousandShips AThousandShips added bug topic:gui cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release labels Jan 22, 2025
@AThousandShips AThousandShips added this to the 4.4 milestone Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release topic:gui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ItemList 'Text Overrun Behavior' doens't work (as expected) ItemList text_overrun_behavior
2 participants