Skip to content

Conversation

@pasitiis
Copy link

I made some changes to how the plugin works with the screen reader.

Enable page browsing with a screen reader when the disablePageInteraction setting is set to false, ensuring consistent behavior with or without a screen reader. The implementation has taken some inspiration from the cookie consent on the gov.uk website.

  • When disablePageInteraction is false, the role is set to "region." Browsing the page is possible with or without a screen reader.
  • When disablePageInteraction is true, the role is set to "dialog." Browsing the page is not possible with or without a screen reader.

Fixed focus shifting in the consentModal and PreferencesModal.

  • The focus shifted multiple times to the consentModal, causing issues for some screen readers. Fixed.
  • PreferencesModal focus is set to the title when the dialog is opened. Focusing on an empty div element added additional noise for some screen readers. Fixed.

Hide svg icon on screen reader.

Tested with VO + Safari / Chrome / Firefox, NVDA + Chrome/Firefox , JAWS + Chrome/Firefox

@vercel
Copy link

vercel bot commented Jul 29, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
cookieconsent-docs ⬜️ Ignored (Inspect) Visit Preview Jul 29, 2025 7:02am

@netlify
Copy link

netlify bot commented Jul 29, 2025

Deploy Preview for cookieconsentv3-playground canceled.

Name Link
🔨 Latest commit 57ee1f0
🔍 Latest deploy log https://app.netlify.com/projects/cookieconsentv3-playground/deploys/688871eb2158da0008f5e493

@stale
Copy link

stale bot commented Aug 28, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 28, 2025
@stale stale bot closed this Sep 5, 2025
@orestbida orestbida reopened this Sep 6, 2025
@stale stale bot removed the stale label Sep 6, 2025
@orestbida orestbida added the triage yet to be reviewed label Sep 6, 2025
@pasitiis
Copy link
Author

pasitiis commented Nov 6, 2025

@orestbida Do you need more explanation on why things have been changed?

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

Labels

triage yet to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants