Skip to content
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

fix(ios-searchbar): fix crash with auto-sized searchbar in iOS11 Fixes #5655 #5658

Merged
merged 2 commits into from
Apr 12, 2018

Conversation

manoldonev
Copy link
Contributor

@manoldonev manoldonev commented Apr 11, 2018

Apparently iOS11 SDK does not support passing sizeThatFits(...) non-finite width value any more.
In a scenario where searchbar should be auto-sized(grid with auto column) our measure logic always passes infinity values in case layout mode is unspecified so we extend the UISearchBar widget and add the fix there.

@manoldonev manoldonev self-assigned this Apr 11, 2018
@ghost ghost added the in progress label Apr 11, 2018
@manoldonev manoldonev force-pushed the mdonev/searchbar-ios-layout-fix branch from 460a4a3 to 0197ee1 Compare April 11, 2018 16:01
@manoldonev manoldonev force-pushed the mdonev/searchbar-ios-layout-fix branch from 0197ee1 to 090b1b5 Compare April 11, 2018 16:07
@manoldonev manoldonev added this to the 4.1 milestone Apr 11, 2018
@manoldonev
Copy link
Contributor Author

@SvetoslavTsenov Added new test page for UI test app ^^^

@manoldonev manoldonev force-pushed the mdonev/searchbar-ios-layout-fix branch from 1058a28 to 1a6fc6e Compare April 12, 2018 08:58
@manoldonev manoldonev merged commit 555e592 into master Apr 12, 2018
@ghost ghost removed the in progress label Apr 12, 2018
@manoldonev manoldonev deleted the mdonev/searchbar-ios-layout-fix branch April 12, 2018 11:13
@lock
Copy link

lock bot commented Aug 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SearchBar component without a width attribute crashes the app without error logs
3 participants