Skip to content

Conversation

@LucioChavezFuentes
Copy link
Contributor

A few changes on forwardedRef allows anyone to pass a React refon Web version and get a ref of <select> tag that renders Picker. I am currently using a forked version with these changes in order to get focus( ) method of <select> tag and avoid dirty code.

Let me know guys what do you think. I am fully available to make any changes neccesary.

@rushatgabhane
Copy link

@Naturalclar gentle bump to review this when you get the time, thanks so much!

Fixes issue #375

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LucioChavezFuentes thanks for the fix!

@Naturalclar
Copy link
Contributor

@LucioChavezFuentes CI seems to be failing could you do me a favor and run the linter on lines you edited?

@LucioChavezFuentes
Copy link
Contributor Author

LucioChavezFuentes commented Feb 21, 2022

@Naturalclar I fixed the lint errors. You can review it again.

@LucioChavezFuentes
Copy link
Contributor Author

And thank you for your time to review.

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LucioChavezFuentes thanks 👍

@Naturalclar Naturalclar merged commit c7e4180 into react-native-picker:master Mar 1, 2022
github-actions bot pushed a commit that referenced this pull request Mar 1, 2022
## [2.3.1](v2.3.0...v2.3.1) (2022-03-01)

### Bug Fixes

* add ref support for Web ([#376](#376)) ([c7e4180](c7e4180))
@Naturalclar
Copy link
Contributor

🎉 This PR is included in version 2.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@LucioChavezFuentes
Copy link
Contributor Author

LucioChavezFuentes commented Mar 1, 2022

Thank you @Naturalclar, I really appreciate your time on review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants