Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a screen reader text option to RichText #57799

Closed
carolinan opened this issue Jan 12, 2024 · 1 comment
Closed

Add a screen reader text option to RichText #57799

carolinan opened this issue Jan 12, 2024 · 1 comment
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Enhancement A suggestion for improvement.

Comments

@carolinan
Copy link
Contributor

carolinan commented Jan 12, 2024

What problem does this address?

There is no easy way for users to add screen reader texts that they can edit in the editor but are visually hidden on the front.
WordPress uses a CSS class, .screen-reader-text to visually hide elements, but even if the user is aware of this feature, this class also hides the element in the editor.

What is your proposed solution?

Add a screen reader text option to RichText.
Just like the RichText toolbar options can be used to wrap text within elements like <strong> and <em> it could be used to wrap text inside a span with a CSS class that only hides the element on the front.

@carolinan carolinan added [Type] Enhancement A suggestion for improvement. [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). labels Jan 12, 2024
@skorasaurus skorasaurus added the [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed label Jan 12, 2024
@skorasaurus
Copy link
Member

Hi @carolinan ; I believe this is being discussed in #22332 please reopen if I misunderstand.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

2 participants