Add optional onPhoneNumberFocus function

johannessjoberg commented 5 years ago

Added optional onPhoneNumberFocus function, similar to onPhoneNumberBlur.

Description

Added onPhoneNumberFocus because it's not possible to extended onFocus in telInputProps property due to onFocus being overwritten in src/components/TelInput.js. There is an open PR addressing the same thing in #239, but it has been stale for some time and lack tests.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have used ESLint & Prettier to follow the code style of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

coveralls commented 5 years ago

Coverage increased (+0.08%) to 93.656% when pulling 0362b6bc18119b3b210d3fa9713a1167715a6f3b on DPOrganizer:feature/on-phone-number-focus into ced4b09318d62c62e33285b215ca0118214c3bf1 on patw0929:master.

patw0929 commented 5 years ago

Thanks a lot! Looks great! 🙌 Could you help to update Documentation - Props?

johannessjoberg commented 5 years ago

Thanks @patw0929! From what I understood the Props storybook automatically picks things from IntlTelInput.propTypes so it should already be in place ☺️

patw0929 commented 5 years ago

You're right! I forgot it 😅 Thank you! Merged & released as v7.1.0.

johannessjoberg commented 5 years ago

Thanks @patw0929 for the fast response and feedback! Great lib! 👍

patw0929 / react-intl-tel-input