Skip to content

Use portaled dropdowns for OSS::CountrySelector & OSS::PhoneNumberInput #491

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

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

Conversation

phndiaye
Copy link
Member

What does this PR do?

Use portaled select dropdowns for both OSS::CountrySelector & OSS::PhoneNumberInput in order to properly:

  • escape overflow hidden traps
  • have autoplacement

What are the observable changes?

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation
  • Migrated touched components to Glimmer Components
  • Properly labeled

Copy link

linear bot commented Apr 11, 2025

@phndiaye phndiaye self-assigned this Apr 11, 2025
@phndiaye phndiaye changed the title Use portaled select dropdowns for both OSS::CountrySelector & OSS::PhoneNumberInput Use portaled dropdowns for both OSS::CountrySelector & OSS::PhoneNumberInput Apr 17, 2025
@phndiaye phndiaye changed the title Use portaled dropdowns for both OSS::CountrySelector & OSS::PhoneNumberInput Use portaled dropdowns for OSS::CountrySelector & OSS::PhoneNumberInput Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants