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-24574] iOS: Fix picker-selection #8958
Conversation
apidoc/Titanium/UI/Picker.yml
Outdated
@@ -353,10 +353,9 @@ properties: | |||
If `true`, selection indicator is enabled. | |||
|
|||
On iOS 7 and later, the picker indicator is always shown and you cannot control it. | |||
|
|||
On iOS 6 and prior, if enabled, a blue bar is displayed to indicate the current selection. | |||
Manually setting the property was deprecated in 6.1.0 and will be ignored by the SDK. |
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 is deprecated in iOS only. So iOS should be mentioned in above line.
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 was written in the iOS-paragraph, so I hoped people will get it's iOS related, but I'll explicitly say it :-).
@@ -190,19 +191,6 @@ -(void)setType_:(id)type_ | |||
} | |||
} | |||
|
|||
// We're order-dependent on type being set first, so we need to make sure that anything that relies | |||
// on whether or not this is a date picker needs to be set AFTER the initial configuration. | |||
-(void)setSelectionIndicator_:(id)value |
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.
Rather deleting the method , we should deprecate it by giving message to developers. What you think?
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 was not used since iOS 7 anyway, so I decided to clean up the core directly. Having it or not does not have an effect to the developer - they cannot disable the selection indicator anyway.
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.
Ft and CR passed.
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.
Approved! picker selection indicator is now visible. Tested on iOS 10.2 / 9.3 Devices and Simulators.
Reverting due to failing unit-tests. |
JIRA: https://jira.appcelerator.org/browse/TIMOB-24574