Skip to content

Conversation

@sogalabhi
Copy link

@sogalabhi sogalabhi commented Dec 26, 2025

Description

This change improves the readability of the statistics dialog by replacing string-based key–value formatting with a structured two-column layout. Keys are rendered in a fixed-width column, ensuring all values align consistently regardless of key length.

Fixes #538

Screenshots

Before
image
After
image

Checklist

  • Tests have been added or updated to cover the changes
  • Documentation has been updated to reflect the changes
  • Code follows the established coding style guidelines
  • All tests are passing

@sogalabhi
Copy link
Author

Wanted suggestions. For now I have hardcoded the width: 180 for the left column. I'll have to get the max width by custom painter or something to know the width the longest string is taking. Shall i do that?

@sogalabhi sogalabhi marked this pull request as draft December 26, 2025 16:50
@sogalabhi sogalabhi marked this pull request as ready for review December 29, 2025 12:35
@sogalabhi
Copy link
Author

@SGI-CAPP-AT2 Kindly review the PR. This and prev one. Both are just minor UI/UX improvements

@SGI-CAPP-AT2
Copy link
Collaborator

@SGI-CAPP-AT2 Kindly review the PR. This and prev one. Both are just minor UI/UX improvements

will it work on any size of device?

@sogalabhi
Copy link
Author

@SGI-CAPP-AT2 Kindly review the PR. This and prev one. Both are just minor UI/UX improvements

will it work on any size of device?

I missed this message. Sorry for late reply. I'll test it and send asap

@sogalabhi
Copy link
Author

@SGI-CAPP-AT2
It works correctly in bigger screens. Have problems with my local to check in emulators. check in real device(tablet). Screenshots:
image
image

@sogalabhi
Copy link
Author

ig we can add a table type for this. It will be more legible. Also I feel I can increase the modal width so that it looks good. Your suggestions?
image

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Misaligned statistics values due to variable key lengths in statistics dialog

2 participants