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

Add iOS support and let parent handle disposal of search box #44

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

alamshafil
Copy link
Contributor

This pull request adds support for iOS (Xcode project file and Pod file).

Additionally, a bug in the iOS build is fixed. The code now ensures proper disposal of searchTermController in home_page.dart and removed unnecessary disposal in search_box.dart.

(This would cause a race-condition between the search box disposing itself since it out of the list view of the parent when scrolling down, but parent does not want to dispose to keep it in the list view.)

@dagmawibabi
Copy link
Owner

@alamshafil this's pretty awesome, can you share a screen recording?

@alamshafil
Copy link
Contributor Author

Apologize for the late response. I have attached the screen recording below. As you can see there is no bug when scrolling down or refreshing.

Simulator.Screen.Recording.-.iPhone.16.Plus.-.2024-12-18.at.22.13.02.mp4

@dagmawibabi
Copy link
Owner

@alamshafil beautiful improvement. This's wonderful, great job! Merged!

@dagmawibabi dagmawibabi merged commit d0a10eb into dagmawibabi:main Dec 19, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants