Skip to content

Conversation

@lumirlumir
Copy link
Member

@lumirlumir lumirlumir commented Oct 10, 2025

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[x] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

This PR is a follow-up to #20181.

In this PR, I've added alternative text for the indicator | and emojis.

image

Previously, screen readers would read out the indicator |, which could be distracting.

(The MDN reference is here: https://developer.mozilla.org/en-US/docs/Web/CSS/content#try_it)

Is there anything you'd like reviewers to focus on?

Ref: #20181

@lumirlumir lumirlumir requested a review from a team as a code owner October 10, 2025 11:30
@eslint-github-bot eslint-github-bot bot added the documentation Relates to ESLint's documentation label Oct 10, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 10, 2025
@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for docs-eslint ready!

Name Link
🔨 Latest commit c53f824
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/68e8ee666806fd000803172a
😎 Deploy Preview https://deploy-preview-20205--docs-eslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Member

@aladdin-add aladdin-add left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@aladdin-add aladdin-add merged commit 462675a into main Oct 11, 2025
31 checks passed
@aladdin-add aladdin-add deleted the docs-improve-web-a11y branch October 11, 2025 06:13
@github-project-automation github-project-automation bot moved this from Needs Triage to Complete in Triage Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Relates to ESLint's documentation

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants