Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Dismiss UISearchController when losing focus #12701

Merged
merged 2 commits into from
Nov 25, 2021

Conversation

kazo0
Copy link
Contributor

@kazo0 kazo0 commented Nov 1, 2020

Description of Change

Dismiss UISearchController when losing focus

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

When a query is confirmed in SearchHandler, the search handler will lose focus and will no longer display a dimmed view on top of the remaining page

Before/After Screenshots

Not applicable

Testing Procedure

  1. Add a Shell SearchHandler
  2. Tap the search bar to focus
  3. Notice the main page is dimmed
  4. Enter a search term
  5. Execute the search
  6. Notice the SearchHandler no longer displays a dimmed background and the main content is tappable

out

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@kazo0 kazo0 force-pushed the sb/ios-search-handler-dismiss branch from 63703c8 to 7727aa6 Compare November 1, 2020 14:37
@kazo0 kazo0 changed the title Sb/ios search handler dismiss Dismiss UISearchController when losing focus Nov 1, 2020
@kalprajkakani
Copy link

Any update?

@kazo0 kazo0 force-pushed the sb/ios-search-handler-dismiss branch 2 times, most recently from e6cfc4e to aba57b0 Compare December 10, 2020 17:11
@varyamereon
Copy link

Is there a plan to get this merged soon? Seems there are a few fixed issues in pull requests that would fix bugs in my app but they are not being merged. Just wondering whether I need to write a workaround or wait.

@demoh2019
Copy link

Any update?

@jsuarezruiz
Copy link
Contributor

@kazo0 Could you rebase your branch to fix the conflicts?. Thanks!

@kazo0 kazo0 force-pushed the sb/ios-search-handler-dismiss branch from 7c469d6 to 093f634 Compare September 17, 2021 12:47
@kazo0
Copy link
Contributor Author

kazo0 commented Sep 20, 2021

@jsuarezruiz done!

@jfversluis
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jfversluis
Copy link
Member

I know, after more than a year, you are still anxiously looking every day at this PR @kazo0, but today is the day it is finally merged. You can now go forth and call yourself a Xamarin.Forms contributor! 🦸

Thank you so much for your time and effort on this and sorry we didn't get to it sooner!

@jfversluis jfversluis merged commit be2471c into xamarin:5.0.0 Nov 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Shell SearchHandler blocks touch to view
7 participants