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

Enable kebab dropdown keyboard accessibility #2535

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

marshmalien
Copy link
Member

@marshmalien marshmalien commented Jun 17, 2024

https://issues.redhat.com/browse/AAP-12914
https://issues.redhat.com/browse/AAP-12952

Update to PatternFly's latest implementation of dropdown and remove deprecated implementation. Some breaking changes addressed:

  • Pass dropdown position as Dropdown popperProps
  • Remove <Tooltip> from dropdown items and pass tooltip properties as DropdownItem tooltipProps
  • Pass dropdown item icon as DropdownItem children
  • Replace <DropdownSeparator> with <Divider>
  • Replace linkable dropdown item component prop with to

keyboard

@github-actions github-actions bot added AWX Label to indicate changes relevant to AWX EDA Label to indicate changes relevant to EDA E2E End-to-end testing HUB CCT Component tests AFW Ansible Framework labels Jun 17, 2024
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch 6 times, most recently from da263c9 to 9d4ee10 Compare June 20, 2024 17:14
@marshmalien marshmalien requested a review from nixocio June 20, 2024 17:15
Copy link
Contributor

@jamestalton jamestalton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome changes. I'd also like to see us fix the overflow scrolling in this!

framework/PageActions/PageActionDropdown.tsx Outdated Show resolved Hide resolved
framework/PageActions/PageActionDropdown.tsx Show resolved Hide resolved
cypress/e2e/awx/administration/instances.cy.ts Outdated Show resolved Hide resolved
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch from 9d4ee10 to afd9c9e Compare July 9, 2024 19:09
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch 7 times, most recently from 0362654 to 63372f6 Compare July 12, 2024 17:45
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch 3 times, most recently from 0657954 to b3cbcf8 Compare July 15, 2024 20:45
jamestalton
jamestalton previously approved these changes Jul 15, 2024
nixocio
nixocio previously approved these changes Jul 16, 2024
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch 2 times, most recently from f687729 to 0a62b90 Compare July 17, 2024 17:54
@marshmalien marshmalien dismissed stale reviews from nixocio and jamestalton via fa69c74 July 17, 2024 19:33
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch 7 times, most recently from 880836c to 887107e Compare July 23, 2024 15:30
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch from 887107e to c785e9b Compare July 23, 2024 15:40
Address e2e and component tests after moving to the latest PatternFly dropdown

Display tooltip for disabled menu toggle page action dropdown
@marshmalien marshmalien force-pushed the keyboard-accessible-dropdown branch from c785e9b to 6567541 Compare July 23, 2024 20:05
@marshmalien marshmalien merged commit 36f29d1 into main Jul 24, 2024
23 checks passed
@marshmalien marshmalien deleted the keyboard-accessible-dropdown branch July 24, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AFW Ansible Framework AWX Label to indicate changes relevant to AWX CCT Component tests E2E End-to-end testing EDA Label to indicate changes relevant to EDA HUB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants