-
-
Notifications
You must be signed in to change notification settings - Fork 669
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
[widget audit] toga.Switch #1832
Conversation
# Allow a class with no new methods to be defined by assigning | ||
# from an existing class. The col_offset means we only pay | ||
# attention to assignments at the top level of a module, not | ||
# assignments inside method bodies. |
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.
This line was catching the old_value = self._get_value('value')
call in the dummy implementation. The line is required to allow for MainWindow = Window
as a top level module assignment to be sufficient to define the existence of an interface class. This isn't a great check - but as soon as we're at 100% testbed coverage, we're not going to need the implementation checks anyway, so I figured a bandaid solution was sufficient.
gtk/tests/widgets/test_switch.py
Outdated
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.
The tests in this module are superseded by the testbed coverage.
Includes a rework of the base
enabled
flag. This was required because Switch on iOS and GTK is a composite widget, so determination of enabled status requires a different implementation.Audit checklist