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-10368] iOS: Expose "color" and "hintTextColor" to Ti.UI.SearchBar #9772
Conversation
apidoc/Titanium/UI/SearchBar.yml
Outdated
description: | | ||
For information about color values, see the "Colors" section of <Titanium.UI>. | ||
type: String | ||
platforms: [iphone, ipad] |
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.
Android already supports this, just wasn't documented. Would you mind adding "android" to the supported platform list please?
apidoc/Titanium/UI/SearchBar.yml
Outdated
- name: hintText | ||
summary: Text to show when the search bar field is not focused. | ||
type: String | ||
default: On iOS, "Search"; on Android, no hint text. | ||
|
||
- name: hintTextColor | ||
summary: Hint text color to display when the field is empty. | ||
platforms: [iphone, ipad] |
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.
Android already supports this property too.
|
||
// TODO: Expose on Android / Windows as well | ||
// We have in in Ti.UI.Android.SearchView for Android, but need more parity here | ||
it.ios('.hintTextColor', function () { |
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.
Android can run this unit test too.
Change to it.windowsMissing
?
}); | ||
|
||
// TODO: Expose on Android / Windows as well | ||
it.ios('.color', function () { |
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.
Android can run this unit test.
Change to it.windowsMissing
?
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.
CR passed.
@jquick-axway Can you update your review? |
Generated by 🚫 dangerJS |
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.
CR: Pass
FR Passed. Tested with 7_1_X branch PR |
JIRA: https://jira.appcelerator.org/browse/TIMOB-10368