-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add settings tab #25
Comments
I like this idea. |
I'm currently working on a settings panel inspired by those found in JetBrains Products. Mind the mess visually and in the code because I've been hacking away at it for about a week or two -- I've changed a lot about its own internal structure and it's still changing as it expands to meet all the needs of evolve and other potential plugins. So far things are going alright. It currently looks like this: It currently supports the following features:
As you could imagine I had to do a lot of heavy lifting around the framework. The nature and complexity of the settings panel demands that ev_menu be a toggle (which I am currently achieving with a bind) for things like string editing or typing in specific values for NumSliders. This strays a bit from the original nature of evolve being just a pop-in, get what you need done, then hide again solution. Unfortunately, I couldn't envision a better way of doing this that didn't place the burden of every plugin implementing its own settings page layout manually. That's why I'm posting this here instead of just presenting a massive pull request later, so that everyone involved can weigh in so that I can make things easier for everyone else involved. |
It contains settings for evolve, rather than something gmod related like the sandbox tab.
Plugins are able to register their own settings.
We might want to group these settings, similar to the permissions on the rank tab.
Plugins may be allowed to register their own groups, as well.
This would be handy for plugins like overwriting playermodel color with rank color, as Frankess suggested.
The text was updated successfully, but these errors were encountered: