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-13419][TIMOB-13421] iOS ListView #4424
Conversation
…w to be visible simultaneously
…s will not show up in search with style plain
…epSectionsInSearch
-(void)configureFooter | ||
{ | ||
if (_footerViewProxy == nil) { | ||
_footerViewProxy = [[TiViewProxy alloc] init]; |
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.
Surely this and the headerView, searchWrapper, etc, can use a factory method for a lot of this. (view proxy alloc init, set layout properties.)
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.
Fixing
@@ -390,6 +551,20 @@ -(void)setPullView_:(id)args | |||
|
|||
} | |||
|
|||
-(void)setKeepSectionsInSearch_:(id)args | |||
{ | |||
[self.proxy replaceValue:args forKey:@"keepSectionsInSearch" notification:NO]; |
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.
Doesn't the proxy do this automatically, with the last action being to call setKeepSectionsInSearch_? In other words, is this line redundant?
-(void)setCaseInsensitiveSearch_:(id)args | ||
{ | ||
caseInsensitiveSearch = [TiUtils boolValue:args def:YES]; | ||
[self.proxy replaceValue:NUMBOOL(caseInsensitiveSearch) forKey:@"caseInsensitiveSearch" notification:NO]; |
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.
shouldn't this be removed too ?
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.
Fixing
Looks good to me. Code Reviewed and APPROVED |
Functionally Tested and APPROVED |
[TIMOB-13419][TIMOB-13421] iOS ListView
Fixes
TIMOB-13419
TIMOB-13421
Use KS PR 106 to test