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

[#218] Fix expanded class not being removed from wrapper #219

Conversation

MilosMosovsky
Copy link
Contributor

As we are attaching global "expanded" class to the wrapper
when countries are open, this class is not being removed once
country is selected and dropdown closed.

Fixes #218

As we are attaching global "expanded" class to the wrapper
when countries are open, this class is not being removed once
country is selected and dropdown closed.

Fixes patw0929#218
@MilosMosovsky MilosMosovsky force-pushed the 218-fix-expanded-class-not-being-removed branch from 5a48d27 to d81a03c Compare June 29, 2018 11:17
@coveralls
Copy link

coveralls commented Jun 29, 2018

Coverage Status

Coverage increased (+0.2%) to 94.186% when pulling d81a03c on MilosMosovsky:218-fix-expanded-class-not-being-removed into b788abe on patw0929:master.

@patw0929 patw0929 merged commit e41c7bb into patw0929:master Jun 29, 2018
patw0929 added a commit that referenced this pull request Jun 29, 2018
* #217: Add findIndex implementation for IE 11
* #219: Fixed #218: Fix expanded class not being removed from wrapper
@patw0929
Copy link
Owner

Merged & published as 5.0.6.
Thank you!

andrewsantarin pushed a commit to andrewsantarin/react-intl-tel-input that referenced this pull request Feb 2, 2022
…lass-not-being-removed

[patw0929#218] Fix expanded class not being removed from wrapper
andrewsantarin pushed a commit to andrewsantarin/react-intl-tel-input that referenced this pull request Feb 2, 2022
* patw0929#217: Add findIndex implementation for IE 11
* patw0929#219: Fixed patw0929#218: Fix expanded class not being removed from wrapper
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.

Class 'expanded' stays on the wrapper when dropdown is closed
3 participants