Skip to content

Conversation

clanps
Copy link

@clanps clanps commented Sep 1, 2025

this is to fix scaling issues on larger displays

Either remove the lines or change them to 100% so the UI scales properly on all displays
in #4626 there were changes made to limit the size, I'm not sure why when the elements don't really spread that far away from each other.
regular examples
before
after
extreme examples
before2
after2


Use the following Checklist if you have changed something on the Backend or Frontend:

  • Tested the feature and it's working on a current and clean install.
  • Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
  • Created / Updated Tests (If necessary)
  • Created / Updated documentation (If necessary)

fix scaling issues on larger displays
@flavioislima
Copy link
Member

flavioislima commented Sep 1, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@clanps
Copy link
Author

clanps commented Sep 1, 2025

I have read the CLA Document and I hereby sign the CLA

@arielj
Copy link
Collaborator

arielj commented Sep 2, 2025

things being too wide hurts readability and accessibility

the left box being too wide makes the paragraphs harder to read https://www.freshconsulting.com/insights/blog/uiux-principle-46-text-box-width-should-help-users-read

and the right box being too wide makes it hard to match the left and right sides of each row together, keeping them close makes it easier to see what's what

that's why the two boxes are kinda limited in width

maybe an option for the left box is to let it grow but have a maximum width for the description to not have that issue, but I think it will look weird

at the end, I'd rather have it look a bit weird for really wide screen rather than having bad readability/accessibility

@clanps
Copy link
Author

clanps commented Sep 2, 2025

I really dislike the boxes not expanding to fill the screen and being glued to the top left margin. I also don't think everyone will suffer from accessibility or readability issues. A toggle that allows this but is off by default would maybe be a better solution than nothing at all?

@arielj
Copy link
Collaborator

arielj commented Sep 2, 2025

there's a Custom CSS feature in general settings > advanced, where you can modify any CSS

@clanps
Copy link
Author

clanps commented Sep 2, 2025

People who also share the same opinion as me shouldn't need to input custom css for something as basic as this. It really should be built in. That seems counterintuitive to being accessible.

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.

3 participants