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

Refactor NewMeetingTeamPicker menu to Radix #9274

Closed
1 task
nickoferrall opened this issue Dec 6, 2023 · 1 comment · Fixed by #9679
Closed
1 task

Refactor NewMeetingTeamPicker menu to Radix #9274

nickoferrall opened this issue Dec 6, 2023 · 1 comment · Fixed by #9679
Assignees
Labels

Comments

@nickoferrall
Copy link
Contributor

The NewMeetingTeamPicker currently has a meeting prop called positionOverride, which feels a bit hacky.

We need to update the positionOverride if it's on mobile. Its position can be unreliable and needs a refresh - see Loom: https://www.loom.com/share/26a4e81d4d2041ca8d4eaff014dd8e0d

Hopefully, refactoring this useMenu to the Radix menu can make it more reliable and enable us to remove the positionOverride prop.

AC

  • Refactor NewMeetingTeamPicker useMenu to the Radix Menu component
@jordanh
Copy link
Contributor

jordanh commented Mar 15, 2024

Going to elevate the priority of this one as this cycle is focusing on getting the AL out to everybody

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Status: No status
Development

Successfully merging a pull request may close this issue.

4 participants