-
Notifications
You must be signed in to change notification settings - Fork 665
Fix radio buttons #15
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
Fix radio buttons #15
Conversation
… for vertices (they check correctly now). Paq.sh now automatically outputs to nuklear.h.
keharriso's Prevent clicking more than one (overlapping) button per frame. However, this has a couple of other changes that should fix dropdowns from intefering as well.
- Sets click to 0 every start of frame - Input was still down for that frame, so it triggered the next event Now it waits for your button to go back up before resetting the clicked variable, so as long as your mouse is down, only one item is clicked.
…checkbox! With README changes that show off more features and the updated style. Slightly modified.
dumblob
left a comment
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.
Please bump version and fix conflicts with other your PRs 😉. Then I'll merge. And thanks again for your effort!
Readme.md
Outdated
| /* init gui state */ | ||
| struct nk_context ctx; | ||
| nk_init_fixed(&ctx, calloc(1, MAX_MEMORY), MAX_MEMORY, &font); | ||
| struct nk_context *ctx = (nk_context*) malloc(sizeof(nk_context)); |
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.
Nitpick: why on heap? It's a small struct and the less allocated memory you have to manage, the better (and more readable).
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.
True, I'll fix this, it's because on my own project there's too much on the stack already.
…nto fix_radio_buttons
|
Updated. |
|
Please bump version also in Btw. why did you remove all the |
|
Will do, the consts are because of the merge with fix_click. The input struct can't be const, since clicking in the mouse rect will modify it (so you can't click through two objects and active both of them). |
…nto fix_radio_buttons
|
Updated. |
…nto fix_radio_buttons
This fix changes the looks of the radio buttons (like vurtun/nuklear#653 zhouxs1023's Modify the appearance for option and checkbox!).
With a few minor changes to make it clearer. And README has been changed accordingly and a more thorough example was added.