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

Fix #6039: splitter bug fixes #6040

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

shawnharvell
Copy link
Contributor

@shawnharvell shawnharvell commented Feb 24, 2024

Fix #6039

for issue #6039

  • move step typing to correct element

  • move role=separator to correct element (per spec should be on the same element that contains aria-orientation and aria-value* attribs) - see https://www.w3.org/TR/wai-aria-1.2/#separator

  • fix bug where if home/end/enter were used in a splitter that has a minSize, after pressing those buttons, the arrow keys would no longer work

for this issue #6039

- move step typing to correct element

- move role=separator to correct element (per spec should be on the same element that contains aria-orientation and aria-value* attribs) - see https://www.w3.org/TR/wai-aria-1.2/#separator

- fix bug where if home/end/enter were used in a splitter that has a minSize, after pressing those buttons, the arrow keys would no longer work
Copy link

vercel bot commented Feb 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
primereact ⬜️ Ignored (Inspect) Visit Preview Feb 24, 2024 0:47am
primereact-v9 ⬜️ Ignored (Inspect) Visit Preview Feb 24, 2024 0:47am

Copy link

Thanks a lot for your contribution! But, PR does not seem to be linked to any issues. Please manually link to an issue or mention it in the description using #<issue_id>.

@melloware melloware added the Component: Accessibility Issue or pull request is related to WCAG or ARIA label Feb 24, 2024
@melloware melloware changed the title splitter bug fixes #6039 Fix #6039: splitter bug fixes Feb 24, 2024
Copy link
Member

@melloware melloware left a comment

Choose a reason for hiding this comment

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

I tested these locally and they look great

@melloware melloware added this to the 10.5.2 milestone Feb 24, 2024
@shawnharvell
Copy link
Contributor Author

@melloware do I need to do anything more at this point, I was assuming it's all in the core team's hands now but thought I should double check. Thanks!

@melloware
Copy link
Member

Yep the core team will review and merge it for me it looks perfect.

@nitrogenous nitrogenous merged commit 5a882d4 into primefaces:master Mar 14, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Accessibility Issue or pull request is related to WCAG or ARIA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Splitter: Accessibility updates
4 participants