-
Notifications
You must be signed in to change notification settings - Fork 484
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
ENH: Add checkBoxControlsButtonToggleState property to ctkCheckablePushButton #908
ENH: Add checkBoxControlsButtonToggleState property to ctkCheckablePushButton #908
Conversation
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.
Could you update the relevant test ?
f3a7c2d
to
6aefd7d
Compare
Thanks @jcfr. Comments addressed and test updated. |
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.
It looks good to me, thanks a lot Kyle, I've just added a minor suggestion for documentation (I am not completely sure that it is correct, so please confirm).
…shButton If checkBoxControlsButtonToggleState is enabled then clicking the checkbox also makes the button pushed (for user's convenience) and unpushing the button makes it unchecked, too (to avoid the ambiguous state when the button's checkbox is checked but the button is not pressed down). The property is disabled for now to preserve the current behavior (for backward compatibility). Co-authored-by: Kyle Sunderland <sunderlandkyl@gmail.com>
6aefd7d
to
fadf76b
Compare
List of changes: ``` $ git shortlog 27bc11d..8ec9136 Jean-Christophe Fillion-Robin (1): COMP: Fix Qt4 build adding fallback implementation for qUtf8Printable() (commontk/CTK#909) Kyle Sunderland (1): ENH: Add checkBoxControlsButtonToggleState property to ctkCheckablePushButton (commontk/CTK#908) ```
If checkBoxControlsButtonToggleState is enabled then clicking the checkbox also makes the button pushed (for user's convenience) and unpushing the button makes it unchecked, too (to avoid the ambiguous state when the button's checkbox is checked but the button is not pressed down).
The property is disabled for now to preserve the current behavior (for backward compatibility).
Co-authored-by: Kyle Sunderland sunderlandkyl@gmail.com
Replaces #899