Skip to content

[test] Add Guidepup screen reader tests#5328

Draft
aarongarciah wants to merge 1 commit into
mui:masterfrom
aarongarciah:evaluate-guidepup-for-base-ui
Draft

[test] Add Guidepup screen reader tests#5328
aarongarciah wants to merge 1 commit into
mui:masterfrom
aarongarciah:evaluate-guidepup-for-base-ui

Conversation

@aarongarciah

@aarongarciah aarongarciah commented Jul 24, 2026

Copy link
Copy Markdown
Member

Adds a minimal Guidepup screen-reader test using the existing Radio fixture, with local VoiceOver support and Windows/NVDA CI coverage. No VoiceOver CI coverage for now.

CI run: https://github.com/mui/base-ui/actions/runs/30082080229/job/89445735040?pr=5328

@pkg-pr-new

pkg-pr-new Bot commented Jul 24, 2026

Copy link
Copy Markdown

commit: eecb32a

@code-infra-dashboard

code-infra-dashboard Bot commented Jul 24, 2026

Copy link
Copy Markdown

Bundle size

Bundle Parsed size Gzip size
@base-ui/react 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 1,223.17 ms -82.47 ms(-6.3%) | Renders: 78 (+0) | Paint: 1,910.36 ms -130.18 ms(-6.4%)

Test Duration Renders
Checkbox mount (500 instances) 69.63 ms ▼-20.18 ms(-22.5%) 1 (+0)
Scroll Area mount (300 instances) 72.13 ms ▼-19.50 ms(-21.3%) 3 (+0)
Tooltip mount (300 contained roots) 49.74 ms ▼-12.70 ms(-20.3%) 1 (+0)

12 tests within noise — details


Check out the code infra dashboard for more information about this PR.

@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit eecb32a
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/6a632ddad01d8c000835b376
😎 Deploy Preview https://deploy-preview-5328--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread pnpm-workspace.yaml
Comment on lines +25 to +28
# Exact versions intentionally updated before the 72-hour release-age window elapsed.
- '@guidepup/guidepup@0.29.2'
- '@guidepup/playwright@0.18.0'
- '@guidepup/setup@0.24.1'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This would be removed before merging.

@@ -0,0 +1,38 @@
import { expect } from '@playwright/test';
import { screenReaderTest as test, type ScreenReaderPlaywright } from '@guidepup/playwright';

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

screenReaderTest automatically uses VoiceOver or NVDA depending on the OS.


const MAX_NAVIGATION_STEPS = 10;

async function navigateToItem(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This helper should be extracted as we see fit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant