Open wojtekmaj opened 1 month ago
I believe that's intentional, because the spread prop could contain an erroneous value of the alt prop. Unfortunately we don't have types available, we are a bit conservative with the approach. Should we let it slide?
Hmmm, eslint-plugin-jsx-a11y does not trigger an error if alt was provided regardless of props spread presence, but I guess from those two behaviors, Biome is preferable by playing it safe. After all that's what linters are for.
Environment information
Rule name
lint/a11y/useAltText
Playground link
https://biomejs.dev/playground/?code=ZgB1AG4AYwB0AGkAbwBuACAATQB5AEkAbQBnAFcAaQB0AGgARQByAHIAbwByACgAewAKACAAIABhAGwAdAAsAAoAIAAgAHMAcgBjACwACgAgACAALgAuAC4AbwB0AGgAZQByAFAAcgBvAHAAcwAKAH0AOgAgAHsAIABhAGwAdAA6ACAAcwB0AHIAaQBuAGcAOwAgAHMAcgBjADoAIABzAHQAcgBpAG4AZwA7ACAAdwBpAGQAdABoADoAIABuAHUAbQBiAGUAcgA7ACAAaABlAGkAZwBoAHQAOgAgAG4AdQBtAGIAZQByACAAfQApACAAewAKACAAIAByAGUAdAB1AHIAbgAgADwAaQBtAGcAIABhAGwAdAA9AHsAYQBsAHQAfQAgAHMAcgBjAD0AewBzAHIAYwB9ACAAewAuAC4ALgBvAHQAaABlAHIAUAByAG8AcABzAH0AIAAvAD4AOwAKAH0ACgAKAGYAdQBuAGMAdABpAG8AbgAgAE0AeQBJAG0AZwBXAGkAdABoAG8AdQB0AEUAcgByAG8AcgAoAHsACgAgACAAYQBsAHQALAAKACAAIABzAHIAYwAsAAoAIAAgAC4ALgAuAG8AdABoAGUAcgBQAHIAbwBwAHMACgB9ADoAIAB7ACAAYQBsAHQAOgAgAHMAdAByAGkAbgBnADsAIABzAHIAYwA6ACAAcwB0AHIAaQBuAGcAOwAgAHcAaQBkAHQAaAA6ACAAbgB1AG0AYgBlAHIAOwAgAGgAZQBpAGcAaAB0ADoAIABuAHUAbQBiAGUAcgAgAH0AKQAgAHsACgAgACAAcgBlAHQAdQByAG4AIAA8AGkAbQBnACAAewAuAC4ALgBvAHQAaABlAHIAUAByAG8AcABzAH0AIABhAGwAdAA9AHsAYQBsAHQAfQAgAHMAcgBjAD0AewBzAHIAYwB9ACAALwA%2BADsACgB9AAoA
Expected result
If props spread does not overwrite
alt
in any way, missingalt
should not be reported.Code of Conduct