Skip to content

Address accessibility feedback#3207

Open
lenacohen wants to merge 3 commits into
masterfrom
accessibility-fixes
Open

Address accessibility feedback#3207
lenacohen wants to merge 3 commits into
masterfrom
accessibility-fixes

Conversation

@lenacohen

Copy link
Copy Markdown
Contributor
  • Added alt text to the EFF logo on the welcome page
  • Added the tracker domain to the "Remove selected" button labels on the Tracking Domains tab of the options page
  • Increased the size of text on orange buttons to meet WCAG standards for color contrast
  • Updated the Privacy Policy text link color to meet WCAG standards for color contrast (in dark mode)

Comment thread src/js/htmlutils.js Outdated
Comment thread src/skin/css/firstRun.css Outdated
}

body, #pb-privacy-policy {
body, #pb-privacy-policy a {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is color: #707070 not too low contrast in light mode also?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should underline the privacy policy link to make it clear it's a link, same as with "EFF" in the header.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It wasn't too low contrast for WCAG level AA, but I'll make it black to meet AAA standards! I also added made the orange links black to meet color contrast requirements and added underlines to make it clear that they're links.

@ghostwords ghostwords added the a11y Accessibility label Jul 16, 2026
Comment thread src/js/htmlutils.js Outdated
@lenacohen
lenacohen force-pushed the accessibility-fixes branch from c47ac2a to da55402 Compare July 20, 2026 17:09
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.

2 participants