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

Settings UI refactoring #269

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

ekomrak
Copy link

@ekomrak ekomrak commented Mar 16, 2024

I would like to suggest the following refactoring of the Settings UI:

  1. Removed a lot of boilerplate code. The number of lines in the source code is reduced.
  2. Now UI is more consistent in terms of Look & Fill.
  3. Now it is fully responsive. Settings window can be resizable if it is necessary.
  4. Simplified readability of code and now it is easy to support and extend settings windows using the same approach.

Few examples:
Current:
General
New:
General

Current:
Graphics_General
New:
Graphics_General

Current:
Graphics_OldEvents
New:
Graphics_OldEvents

Current:
Graphics_CinemaMode
New:
Graphics_CinemaMode

Current:
Sounds
New:
Sounds

xspanger3770 and others added 6 commits February 23, 2024 19:23
Finally update main branch with 0.11.x version features (336 commits) so that my github profile page lights up like a christmas tree and I can start updating the readme files
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.

None yet

2 participants