-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 focus support for checkbox #287
Conversation
This commit updates the checkbox widget to receive focus implementing the focusable interface. This will allow: - to receive focus via Tab and Shift Tab - to be checked and unchecked using the "Space" key on desktop when focused
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.
Looking good. 2 issues outside the comments inline:
- The focus code in internal/driver/focus.go should probably be checking Disablable as you can currently focus disabled widgets and I think this is the first time that was the case.
- The focus colour is no longer applicable because the current one is used for marking focus on background objects - but these are foreground. I am not sure if we should have 2 shades of focus colour, change the current one, or mark the focusing in a different way?
This is looking great. I think there is just the comment about hover vs focus colour that I know you are already working on. Will come back to it later. |
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.
Great, thanks
This commit updates the checkbox widget to receive focus implementing the Focusable interface.
This will allow: