Skip to content

fix: add accessible names to wallet controls - #253

Merged
restorenode merged 1 commit into
mainfrom
fix/a11y-accessible-names
Aug 13, 2026
Merged

fix: add accessible names to wallet controls#253
restorenode merged 1 commit into
mainfrom
fix/a11y-accessible-names

Conversation

@restorenode

@restorenode restorenode commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Accessibility Improvements
    • Added descriptive labels to quantity, slippage, and operation controls.
    • Improved screen reader identification for wallet drawers and modal dialogs.
    • Enhanced accessibility across bridge and wallet interfaces.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9306942f-9ab2-4ae5-8bfc-d14f198a8ccd

📥 Commits

Reviewing files that changed from the base of the PR and between f8bb881 and 8c5fad6.

📒 Files selected for processing (5)
  • packages/interwovenkit-react/src/components/form/QuantityInput.tsx
  • packages/interwovenkit-react/src/pages/bridge/OperationItem.tsx
  • packages/interwovenkit-react/src/pages/bridge/SlippageControl.tsx
  • packages/interwovenkit-react/src/public/app/Drawer.tsx
  • packages/interwovenkit-react/src/public/app/Modal.tsx

Walkthrough

The changes add accessible names to bridge controls, quantity and slippage inputs, and wallet drawer and modal popups.

Changes

Accessibility Labels

Layer / File(s) Summary
Control accessible names
packages/interwovenkit-react/src/components/form/QuantityInput.tsx, packages/interwovenkit-react/src/pages/bridge/SlippageControl.tsx, packages/interwovenkit-react/src/pages/bridge/OperationItem.tsx
Quantity, slippage, and operation controls now include accessible labels.
Wallet popup accessible names
packages/interwovenkit-react/src/public/app/Drawer.tsx, packages/interwovenkit-react/src/public/app/Modal.tsx
The drawer and modal popups now use the accessible name Initia wallet.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Mergeability Score: ⚪ Minimal · up to 8c5fa

This PR adds accessible names to wallet controls through small, localized UI changes. No actionable merge-blocking risk remains, so it is merge-ready after normal checks and review.

Suggested reviewers: simcheolhwan

Poem

A rabbit hops through labels bright,
Making controls clear in sight.
“Amount,” “Slippage,” names now glow,
Wallet popups speak what they show.
Accessibility takes flight!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the primary change: adding accessible names to wallet controls.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/a11y-accessible-names

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying interwovenkit with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c5fad6
Status: ✅  Deploy successful!
Preview URL: https://ab639e96.interwovenkit.pages.dev
Branch Preview URL: https://fix-a11y-accessible-names.interwovenkit.pages.dev

View logs

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying interwovenkit-testnet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c5fad6
Status: ✅  Deploy successful!
Preview URL: https://fb8c6a32.interwovenkit-testnet.pages.dev
Branch Preview URL: https://fix-a11y-accessible-names.interwovenkit-testnet.pages.dev

View logs

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying interwovenkit-staging with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c5fad6
Status: ✅  Deploy successful!
Preview URL: https://20e8e98d.interwovenkit-staging.pages.dev
Branch Preview URL: https://fix-a11y-accessible-names.interwovenkit-staging.pages.dev

View logs

@restorenode
restorenode merged commit 5806d0a into main Aug 13, 2026
8 checks passed
@restorenode
restorenode deleted the fix/a11y-accessible-names branch August 13, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants