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
Create overlays dropdown #19895
Create overlays dropdown #19895
Conversation
Height: 25 | ||
Text: Buildable | ||
Font: Bold | ||
Key: f2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't reimplement hotkeys in this pr. They need to be handled differently not just because of the dropdown works, but also because they aren't editable by users in the hotkeys menu
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the hotkey just because everything else had one, but I don't think this requires a quick keyboard access.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think having those on hotkeys could lead to a smoother workflow. Regardless I believe we should have a dropdown class and unhardcode values into yaml. That could also allow for a smoother reimplementation of hotkeys
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but that would be worthy of a separate pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks a bit odd when only one overlay trait is present (BuildableTerrainOverlay
is optional), but if a mod only has one overlay it shouldn't use a dropdown for it anyway. (Besides it seems you already have another option coming up.)
I think it makes sense to group overlays, I made Grid and Buildable into a dropdown. This is also future proofing incase we want to add more overlays