Skip to content

fix: associate reflection fields with labels - #186

Open
Aayush-Raj07022008 wants to merge 1 commit into
NST-DEVFORGE:mainfrom
Aayush-Raj07022008:fix/accessibility-reflection-fields
Open

Aayush-Raj07022008 wants to merge 1 commit into
NST-DEVFORGE:mainfrom
Aayush-Raj07022008:fix/accessibility-reflection-fields

Conversation

@Aayush-Raj07022008

Copy link
Copy Markdown

What

Fixed the accessibility of the reflection fields by associating each label with its textarea and connecting the word-count hint using aria-describedby.

Fixes #120

How I tested it

  • npm run lint — passed with 0 errors
  • npm run build — passed
  • git diff --check — passed

Checklist

  • I was assigned the issue before I started
  • npm run lint and npm run build pass locally
  • The diff only touches what the issue asked for

@vercel

vercel Bot commented Oct 4, 2026

Copy link
Copy Markdown

@AayushGH is attempting to deploy a commit to the Geetansh Goyal's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown

📸 How this looks

Built from this PR, so you can see the change without checking it out.

/

Desktop (1280px), top 3000px
Mobile (390px), top 3000px

/learn/open-source

Desktop (1280px), top 3000px
Mobile (390px), top 3000px

Screenshots are taken from a production build of this PR, signed out. Pages behind sign-in show the signed-out view. Build log

github-actions Bot added a commit that referenced this pull request Oct 4, 2026

This branch has not been deployed

No deployments
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.

Workbook reflection: textarea labels aren't connected to their textareas

1 participant