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

POC - auro-popover: Native support #239

Closed
braven112 opened this issue May 9, 2024 · 3 comments
Closed

POC - auro-popover: Native support #239

braven112 opened this issue May 9, 2024 · 3 comments
Assignees

Comments

@braven112
Copy link
Member

braven112 commented May 9, 2024

Is your feature request related to a problem? Please describe.

We would like to incorporate the new native HTML version of popover. We need to first identify the scope of this work. What components will end up changes as a result of this.

We have had a number of issues where the popover doesn't appear where it should and our hypothesis is that leveraging the native popover will be more reliable and allow us to get rid of popover.js

Describe the solution you'd like

This component is either retired and replaced with native support or this component is updated to utilize native html support.

Describe alternatives you've considered

No response

Additional context

This component is often used in auro-dialog for form components and it is in those state where we see the majority of issues.

Exit criteria

  1. a POC version of Auro-Popover has been created that uses native Popover to validate that it will fit our needs
  2. Nees to show using a popover in full screen desktop, mobile, and in a bib.
  3. The POC is not complete until we also outline what the keyboard accessibility experience is out of the box.
  4. Anything that native popover doesn't do, but popperJS does do (that we use) is documented and brought to the team for discussion about how we want to approach that issue.
  5. Determine how Native popover will impact the mobile experience and if it will be suitable for mobile
@braven112 braven112 added Type: Feature New Feature not-reviewed Issue has not been reviewed by Auro team members labels May 9, 2024
@Patrick-Daly-AA Patrick-Daly-AA removed their assignment May 14, 2024
@Patrick-Daly-AA Patrick-Daly-AA removed the not-reviewed Issue has not been reviewed by Auro team members label May 14, 2024
@Patrick-Daly-AA Patrick-Daly-AA changed the title Spike - auro-popover: Native support POC - auro-popover: Native support May 14, 2024
@jordanjones243 jordanjones243 self-assigned this Sep 3, 2024
@jordanjones243 jordanjones243 transferred this issue from AlaskaAirlines/auro-popover Sep 3, 2024
@Patrick-Daly-AA Patrick-Daly-AA removed their assignment Sep 4, 2024
@jordanjones243
Copy link
Contributor

The outcome of this POC was that it is not what we want to use to replace PopperJS because of the lack of positioning API needed to achieve how we want the component to behave. I am not closing this out yet, but moving back to investigating, as there was some discussion about using the modal dialog features provided in Native Popover in order to achieve the new dropdown mobile look.

@jordanjones243 jordanjones243 removed their assignment Sep 13, 2024
@Patrick-Daly-AA
Copy link

Native Popover will not meet our needs. We are investigating Floating UI instead.

@jason-capsule42
Copy link
Member

Duplicate of #218

@jason-capsule42 jason-capsule42 marked this as a duplicate of #218 Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants