Skip to content

fix: add better aria-label to close button#6241

Open
nicodh wants to merge 1 commit intomainfrom
remove-quote
Open

fix: add better aria-label to close button#6241
nicodh wants to merge 1 commit intomainfrom
remove-quote

Conversation

@nicodh
Copy link
Copy Markdown
Member

@nicodh nicodh commented Apr 14, 2026

related to #4611

Copy link
Copy Markdown
Member

@WofWca WofWca left a comment

Choose a reason for hiding this comment

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

I'm not sure if this is really an improvement if we're not translating the strings. People can at least sort of infer the meaning of "close" given the respective section label, as a sighted user would understand the meaning of a cross. But if they don't speak English the button is gonna be a mystery to them.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants