Skip to content

Qt: Group OSD Checkboxes - #12979

Merged
lightningterror merged 1 commit into
PCSX2:masterfrom
RedDevilus:MinorQtWork23
Jul 8, 2025
Merged

Qt: Group OSD Checkboxes#12979
lightningterror merged 1 commit into
PCSX2:masterfrom
RedDevilus:MinorQtWork23

Conversation

@RedDevilus

@RedDevilus RedDevilus commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

Description of Changes

Image

Done some preliminary work on grouping all the OSD checkboxes including the new ones in a better to glance 3 column viewscreen.

Split from #12642 which had system time and date on top

Rationale behind Changes

Easier to group and glance at, ADHD brain doesn't help.

Currently on master looks like this:

Master

vs

PR:

PR

Suggested Testing Steps

Click buttons.

Did you use AI to help find, test, or implement this issue or feature?

No.

Done some preliminary work on grouping all the OSD checkboxes including the new ones in a better to glance 3 column viewscreen.

Split from PCSX2#12642 which had system time and date on top
@github-actions github-actions Bot added the GUI/Qt label Jul 6, 2025
@RedDevilus

Copy link
Copy Markdown
Contributor Author

Image

@TheTechnician27

TheTechnician27 commented Jul 6, 2025

Copy link
Copy Markdown
Contributor

Proposed OSD Settings Grid

With not a lot of used space in the center column, it seems like the Warnings for User group is better-placed underneath the System Information one. This is even accounting for #12642, as it still better balances the column lengths.

@RedDevilus

Copy link
Copy Markdown
Contributor Author

Proposed OSD Settings Grid

With not a lot of used space in the center column, it seems like the Warnings for User group is better-placed underneath the System Information one. This is even accounting for #12642, as it still better balances the column lengths.

It's going to be padded out and also the issue is that people will tend to uncheck it from middle column, but I do get it's less crowded in middle section.

@RedDevilus RedDevilus added this to the Release 2.6 milestone Jul 7, 2025
@dezraj

dezraj commented Jul 8, 2025

Copy link
Copy Markdown

Hello developers! I have some suggestions for this modification.

1- In "Settings and Inputs", perhaps you could briefly clarify which "indicators" and "settings" are shown when we tick those options. Currently they look generic, and a little confusing because there are other options related to "settings". I am a little acquainted with emulators and I actually don't know what "settings" and "indicators" in those options really refer to, imagine someone who is using PCSX2 for the first time...

2- Maybe you could rearrange the "Warn about unsafe settings" section in a different way, like this:
462946343-06b2443c-a970-4bd7-8199-fb4c7234fb34 edited3

I relocated it to "Settings and Inputs" since it is an option related to settings. I also renamed it using "show" to keep syntactic parallelism with the other settings. Finally, I placed this column in the middle to balance the visual difference between the three sections. Now the columns are organized from the most populated to the least one.

This organization may potentially draw more attention to the "System Information" column, since it is the smaller column and placed in the border look more branched from the others.

But the "unsafe settings" warning could also be placed in the last column, like this:
462946343-06b2443c-a970-4bd7-8199-fb4c7234fb34 edited4

Anyway, I hope those suggestions can be of any use.

Thank you for your work.

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.

8 participants