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

Fix flags for firefox and safari #116

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mckelveygreg
Copy link

@mckelveygreg mckelveygreg commented Nov 12, 2021

Fixes 2 bugs:

I was not able to run this locally, but I tested by using these overrides in my code 🤷
Webpack wasn't able to find @mui/styles and react

@mckelveygreg
Copy link
Author

@fnowacki Is there anything else I need to do for this to be merged?

@david-vendel
Copy link

Hello @mckelveygreg thank you for this fix. It indeed works well.
Seem like noone is maintaining this library anymore, and owners don't even care about it to give a merge permission for this repository to somoene from us.
So I got fed up, and since I need this to be fixed for my client project, I forked this library, fixed this bug and published it at:
https://github.com/justdvl/material-ui-phone-number-2
https://www.npmjs.com/package/material-ui-phone-number-2

I recommend everyone who needs this library but working, to make a simple switch. I will also actively maintain this version 2 library, I will make few more improvements in following days

@viclafouch
Copy link

This package (https://github.com/viclafouch/mui-tel-input) handles this problem, supports both React 17 / 18

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.

4 participants