-
Notifications
You must be signed in to change notification settings - Fork 85
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
DropDownOption #1512
DropDownOption #1512
Conversation
I am yet to test this. |
@kianzarrin Could you also take a look at the tooltip for sliders at some point (out of scope for this PR): Should show current slider value but CO UI is very unreliable at updating the tooltip. Alternative would be custom slider component eg. something like Quistar's Tree Anarchy mod? |
3fed3d9
to
c7dc874
Compare
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.
Also noticed 2 drop-downs not yet converted in GeneralTab_LocalisationGroup.cs
:
- language
- road sign theme
They where not covered by OptionsManager so fixing them does not contribute toward #363. they don't have enum so they may need special code so lets deal with them in another 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.
LGTM 👍
Built from source as DEBUG and RELEASE. Tested from main menu, and in-game.
bumping for great justice |
fixes: #1510 . No functional changes.
DropDownOption
that can handle any integerenum
(maybe I should assert).KeyAttribute
that allows us to put the translations directly on the enum members.