fix: add counter to aria-describedby for screen reader accessibility #8859
Conversation
|
🚀 MCP preview deployed to Vercel: https://kolibri-3ki2b9ybk-public-ui-kolibri-mcp.vercel.app |
|
Netlify Draft Deployment |
|
@yurtsever - bitte erst Pipelines grün vor Review |
…s://github.com/public-ui/kolibri into merge-screen-reader-accessibility-for-textarea
deleonio
left a comment
deleonio
left a comment
There was a problem hiding this comment.
Hier sind jede Menge Snaps wegen den Errors, aber nicht ein einziger wegen des Counters. Oder?
| _id: string; | ||
| _touched?: TouchedPropType; | ||
| _hideMsg?: boolean; | ||
| _hasCounter?: boolean; |
There was a problem hiding this comment.
Wozu? Wird nicht benutzt,
There was a problem hiding this comment.
Ich habe mir das mal angeschaut. das wird benutzt. Das ist ja in dem state Objekt welches als argument übergeben wird. (state._hasCounter)
| ariaDescribedBy.push(`${state._id}-hint`); | ||
| } | ||
|
|
||
| if (state._hasCounter) { |
There was a problem hiding this comment.
Hier verwendest Du plötzlich den State?
| ariaDescribedBy.push(`${state._id}-counter`); | ||
| } | ||
|
|
||
| if (hasError === true) { |
There was a problem hiding this comment.
Das war nicht drin in v3
No description provided.