-
Notifications
You must be signed in to change notification settings - Fork 450
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 disabled status to FlxButton #3083
Comments
* Add disabled status to FlxButton (#3083) * create FlxButtonState enum * fix unit tests --------- Co-authored-by: George FunBook <[email protected]>
Would it be a good idea to add a disabled state to more flixel ui stuff |
We discuss this in #3087. The changes are being made here: itulau/flixel-ui@601bf1f |
Yes, https://github.com/itulau/flixel-ui/tree/flx-button-disabled contains my latest changes which should probably remove some issues related to adding the new disabled status. What I'm missing now is a demo to test those changes, which is becoming a bit tricky since I don't use flixel-ui's classes that much so I'm not sure where to begin. |
Try some of the demos: Also, in your branch, I'm wondering if the toggle button graphic should have 2 disabled states, seeing how it has 2 of every other kind? |
A very common feature that is missing from FlxButton which I think would be beneficial to have: A DISABLED state for FlxButton which would make the button ignore inputs and be displayed as being unable to be clicked (greyed out).
The text was updated successfully, but these errors were encountered: