-
Notifications
You must be signed in to change notification settings - Fork 53
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
ConfigEntry<Color> suggestion #76
Comments
Any ideas on how to do the UI to allow both sliders and hex value edits? |
I created a Harmony plugin that mods ConfigurationManager to add some custom handing for the default drawers built in. Other changes in the mod include invalid float value handling and enabling wordWrap on string-type ConfigEntry text fields. I'd be happy to see any of these incorporated into the official ConfigurationManager. |
If you could make a PR with the changes then I would definitely merge it. |
Uh I'll give it a shot, it might take awhile since my scope was limited to Valheim modding. Are there any guidelines/styling/etc to read and follow for PRs? |
It shouldn't be all that different, it doesn't look like it would need to use anything game-specific from the screenshot.
Not really, just the usual. Try to follow the current code style and avoid changing more than necessary. |
The ConfigEntry configs only use sliders. It would be nice if there was an inputfield for editing the color's hex code.
The text was updated successfully, but these errors were encountered: