feat - UI Scaling (Padding / Font Size / Screen Scale) #19233
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
noeliaSD
requested review from
Rsttskyy,
alexjba,
jrainville,
micieslak and
sunleos
Member
Jenkins BuildsClick to see older builds (52)
|
Collaborator
|
Great job, @noeliaSD! looking forward to trying it out my Android device. |
noeliaSD
force-pushed
the
feat/issue-scalingTest
branch
2 times, most recently
from
34c7c4c to
9d0c8af
Compare
noeliaSD
changed the title
Contributor
Author
|
I modified the default values for small screen sizes as discussed:
Padding sliders is kept on the Now the PR is ready to be merged! |
noeliaSD
force-pushed
the
feat/issue-scalingTest
branch
from
9d0c8af to
54114c5
Compare
micieslak
approved these changes
Nov 17, 2025
Member
micieslak
left a comment
micieslak
left a comment
There was a problem hiding this comment.
Clean and working nice! Thx!
alexjba
reviewed
Nov 17, 2025
Contributor
alexjba
left a comment
alexjba
left a comment
There was a problem hiding this comment.
Nice work! Minor things below
noeliaSD
force-pushed
the
feat/issue-scalingTest
branch
from
54114c5 to
4dda617
Compare
Contributor
Author
|
@alexjba, updated according to your suggestions! |
…t scaling factors for short screens - Adds a new padding factor slider to the `AppearanceView` to experiment with UI scaling behavior. This allows adjusting paddings dynamically for testing different density levels and layout responsiveness. - Adds a new 100% screen scaling factor for mobile to provide a more natural and consistent visual baseline across devices. Also updates the default UI configuration: - Font size: set to S - Padding: set to XXS These defaults help achieve a cleaner, more compact layout while improving readability and overall UI density on mobile.
noeliaSD
force-pushed
the
feat/issue-scalingTest
branch
from
4dda617 to
5105cfc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #19219
Exploration Summary
This task was focused on exploring how different combinations of padding, font size, and screen scale affect the overall look and feel of the mobile app.
By exposing these three parameters through sliders, we were able to quickly test various density and text sizes in real time.
Key Findings
QT_SCALE_FACTORcan further enhance visual proportions, especially on high-density displays — although it requires restarting the app.My current preferred setup:
This combination seems to provide the best balance between readability, spatial density, and overall visual comfort on mobile.
Next Steps
master.Notes
This exploration serves as a quick prototyping tool to better understand scaling dynamics on mobile and to identify improvements in perceived density, readability, and overall visual balance.