[Blur & Focus] Support relinquishing focus from multiple views via co…

[josebalius]

…ntainer - #113

This adds the ability to call this.refs.parentView.blur() and blur any active text fields inside the view. It will first try to blur the field (already supported) and if the field is not found as an "active" text field, it will transverse the view and resignFirstResponder for each of them.

I am currently using it in my app and works well.

[ide]

You should be able to do something like [subview endEditing:YES] instead of doing the complete traversal. Also can remove the uiManager parameter since self == uiManager.

UIView *view = uiManager.viewRegistry[reactTag];
if (view) {
  [view endEditing:YES];
} else {
  // Recurse through the shadow hierarchy when the shadow view has no backing UIView
  for (RCTShadowView *subview in [shadowView reactSubviews]) {
    [self endEditingForShadowView:subview.reactTag viewRegistry:viewRegistry];
  }
}

[josebalius]

@ide Makes perfect sense, i'll make the changes, test and commit.

[josebalius]

@ide Made the changes you requested and tested them in my app and everything seems to work great, let me know if you think anything else is needed.

[brentvatne]

I like this 😄 👍

[josebalius]

@brentvatne not sure what went wrong with the build?

[brentvatne]

@josebalius - if #113 works (assuming we can resolve the issue you mentioned in #113 (comment)) is this still needed?

[facebook-github-bot]

@josebalius updated the pull request.

[ghost]

@spicyj would you mind taking a look at this pull request? It's been a while since the last commit was reviewed.

[mkonicek]

@josebalius Can you please indicate whether this PR is still needed? See the comment by Brent Vatne:

if #113 works (assuming we can resolve the issue you mentioned in #113 (comment)) is this still needed?

[ghost]

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @nicklockwood as a potential reviewer. Could you take a look please or cc someone with more context?

[josebalius]

I think this can probably be closed guys.

[mkonicek]

Thanks for looking @josebalius. I assume this was resolved in #113?

We are exposing TextInputState in #3308 which will allow you to do this too. I'm not sure it's worth exposing this one line function as it will increase API surface area ...