Skip to content

fix(web): use button for provider tool item - #40362

Open
lyzno1 wants to merge 1 commit into
mainfrom
codex/a11y-provider-tool-button
Open

fix(web): use button for provider tool item#40362
lyzno1 wants to merge 1 commit into
mainfrom
codex/a11y-provider-tool-button

Conversation

@lyzno1

@lyzno1 lyzno1 commented Aug 9, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the provider tool detail card with a native type=button
  • use the localized tool label as its accessible name
  • expose unavailable state through disabled
  • verify Enter opens the detail and disabled items do not
  • remove the two matching suppressions

Visual regression review

The card remains full width with identical padding, radius, border, background, shadow, hover, two-line clamp, disabled opacity, and cursor. appearance-none and text-start preserve platform chrome and logical alignment. The only intended new state is the keyboard focus-visible ring.

Verification

  • owner suite: 2/2 passed
  • standalone a11y lint: 0 diagnostics
  • full static check: 0 errors
  • suppression delta: -2
  • diff check: passed

Dependency and rollback

Independent root layer; no settings form or icon refactor is included.

@lyzno1 lyzno1 changed the title codex/a11y provider tool button fix(web): use button for provider tool item Aug 9, 2026
@github-actions github-actions Bot added the web This relates to changes on the web. label Aug 9, 2026
@codecov

codecov Bot commented Aug 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.43%. Comparing base (7522ae1) to head (99b9861).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #40362      +/-   ##
==========================================
- Coverage   86.47%   86.43%   -0.05%     
==========================================
  Files        5264     5107     -157     
  Lines      294867   290740    -4127     
  Branches    59029    57923    -1106     
==========================================
- Hits       254976   251289    -3687     
+ Misses      34950    34510     -440     
  Partials     4941     4941              
Flag Coverage Δ
cli ?
dify-ui 93.49% <ø> (ø)
web 86.11% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@lyzno1
lyzno1 force-pushed the codex/a11y-provider-tool-button branch from bf281d4 to 85535bb Compare August 9, 2026 13:47
@lyzno1
lyzno1 force-pushed the codex/a11y-provider-tool-button branch from 85535bb to 99b9861 Compare August 9, 2026 13:48
@lyzno1
lyzno1 marked this pull request as ready for review August 9, 2026 15:47
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant