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
TIMOB-8835: BlackBerry: Implement UI.Switch #58
TIMOB-8835: BlackBerry: Implement UI.Switch #58
Conversation
Reviewers: JP, DavidL ChangeLog: - Implemented native ToggleButton class - Implemented checked property, methods - Updated yaml doc for ToggleButton Test Cases: - Build tibb. - Build tibbtest. - In the java script, instance switch object. - Make sure existing property checked (value) works. - Run docgen.py script - Make sure documentation includes appropriates changes for blackberry platform NOTE: in validate.py script's supported platforms sections blackberry platform added as a part of another PR
@@ -11,6 +11,8 @@ description: | | |||
On Mobile Web, a switch always has text associated with it, and appears as a | |||
toggle button, similar to Android. | |||
|
|||
On BlackBerry, a switch is a fixed-size switch control with two states: checked or unchecked. |
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 description should be more along the lines of what is done for the other platform. Specifically for the switch, the main difference among platforms seems to be the support for associated text with the switch.
So the description needs to cover that, does the BB toggle button support associated text like android and mobileweb or does it not, like iOS?
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.
Updated.
Reviewed |
…rry' into switchBranch #Some more info....
Updated. |
Reviewers: JP, DavidL ChangeLog: - Updated yaml doc for ToggleButton. - static functions names in TiUIObject changed to prefix underline. Test Cases: - Build tibb. - Build tibbtest. - In the java script, instance switch object. - Make sure existing property checked (value) works. - Run docgen.py script - Make sure documentation includes appropriates changes for blackberry platform NOTE: in validate.py script's supported platforms sections blackberry platform added as a part of another PR
Thanks Alex, Approved |
Approved |
…rry' into switchBranch Conflicts: blackberry/tibb/NativeContainerObject.cpp blackberry/tibb/NativeObject.h blackberry/tibb/NativeObjectFactory.cpp blackberry/tibb/TiUIObject.cpp blackberry/tibb/TiUIObject.h
Merged with upstream |
TIMOB-8835: BlackBerry: Implement UI.Switch
Reviewers: JP, DavidL
ChangeLog:
Test Cases:
NOTE: in validate.py script's supported platforms sections blackberry platform added as a part of another PR