Skip to content

Rework the preferences dialog and keyboard shortcuts - #9066

Open
Bouh wants to merge 6 commits into
masterfrom
preferences
Open

Rework the preferences dialog and keyboard shortcuts#9066
Bouh wants to merge 6 commits into
masterfrom
preferences

Conversation

@Bouh

@Bouh Bouh commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Try it:
http://gdevelop-storybook.s3-website-us-east-1.amazonaws.com/preferences/latest/index.html?path=/story/preferences-preferencesdialog--web

  • Rework the preferences that was small and messy.
  • Storybook web & desktop are tested.
  • Test the change in the app manually:
    • New tilemap shortcuts based on well-know layout of existing tilemap editor
    • 3D camera hardcoded wasd, can be change, tested with zqsd
    • Reset all the shortcuts
    • Reset layouts
image image image image

@Bouh
Bouh requested a review from 4ian as a code owner September 6, 2026 22:27
@4ian

4ian commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Surely a matter of taste, but I don't find the alternating row colors nice:
image

It's not super pretty, and looks a bit weird. We don't do that in the rest of GDevelop apart for tables, but here it's more a list of "properties" which don't really go well with this?

@Bouh

Bouh commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator Author

Surely a matter of taste, but I don't find the alternating row colors nice:
It's not super pretty, and looks a bit weird. We don't do that in the rest of GDevelop apart for tables, but here it's more a list of "properties" which don't really go well with this?

It wasn't easy to see how the buttons/toggles lined up with the descriptions, but that must be because of the time鈥攍ate at night, it's easier on the eyes to follow a line with a background color. I'll remove the colored backgrounds tomorrow.

@Bouh

Bouh commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator Author

I removed the background colors and tested a few parts of the reworked interface. See the updated first post.

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.

2 participants