📎 Improve diagnostic of `useValidAriaProps`

biomejs / biome

A toolchain for web projects, aimed to provide functionalities to maintain them. Biome offers formatter and linter, usable via CLI and LSP.

https://biomejs.dev

Apache License 2.0

13.94k stars 422 forks source link

📎 Improve diagnostic of `useValidAriaProps` #364

Closed ematipico closed 11 months ago

ematipico commented 11 months ago

Description

Rule https://biomejs.dev/linter/rules/use-valid-aria-props

We have few options here, we can implement all of them or some of them:

provide a note like "Did you mean ... ?"
maybe add a unsafe code action that removes it

vasucp1207 commented 11 months ago

Can we put a list of all valid aria props on the website and link to that in diagnostic?

ematipico commented 11 months ago

How long is the list of aria properties? I think we apply your suggestion. The list of aria properties rarely changes, so I think we can have a hardcoded list in the rule's documentation page.

vasucp1207 commented 11 months ago

Here 48 in total