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-25850] : iOS SearchBar doesn't show Bookmark button when set on creation in TableView #9932
Conversation
…on creation in TableView
Tests:
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.
Please loop through the proxy keys instead of checking all manually.
…n creation in TableView
|
||
// Set search bar properties to new search bar | ||
NSDictionary *properties = [self allProperties]; | ||
for (NSString *key in properties.allKeys) { |
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.
I assume [self allProperties]
(dynprops
) cannot be nil
, otherwise the app could crash here.
@vijaysingh-axway Do we have tests for all properties of the search-bar already? If not, please add them. We should get too easy with setting the |
@hansemannn In TIUISearchBar, getter is not implemented properly. It simply returns the value which is set in proxy. Ideally it should return value by fetching from searchbar. Currently whatever you set the value of property, it will return same even though that value is not passed to searhbar. So I am thinking to create a ticket for that, in which we can add all these property in unit tests as well. Thats why I have not added unit test in this ticket. Lets discuss on this. |
Like it, please do so! Maybe for the 7.3.0 timeline then. |
Closing in favor of #9947 |
https://jira.appcelerator.org/browse/TIMOB-25850