Skip to content

Issue pinata health roving tabindex - #745

Merged
king-aj-the-first merged 2 commits into
OpenLedger-Foundation:mainfrom
jhayniffy:issue-pinata-health-roving-tabindex
Aug 31, 2026
Merged

Issue pinata health roving tabindex#745
king-aj-the-first merged 2 commits into
OpenLedger-Foundation:mainfrom
jhayniffy:issue-pinata-health-roving-tabindex

Conversation

@jhayniffy

Copy link
Copy Markdown
Contributor

closes #717
closes #719
closes #720
closes #721

…load

usePinataHealth is already wired into the create wizard's upload step
(banner, retry, aria-live, disabled input) — annotate the existing
implementation to make the coverage explicit.
…onTable rows

Rows were mouse-only focusable, so keyboard users couldn't operate the
invoice comparison grid. Wire useRovingTabIndex so arrow/Home/End move
a single tab stop across rows, and announce the focused row via an
aria-live region for WCAG keyboard operability.
@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@jhayniffy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@king-aj-the-first
king-aj-the-first merged commit 91e099b into OpenLedger-Foundation:main Aug 31, 2026
9 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants