Skip to content

FOLLOW-UP: Add aria-label to bulk-update per-row Active checkboxes (from PR #59) #65

@adnaan

Description

@adnaan

Context

This follow-up task was identified during the review of PR #59.

Source PR: #59
PR Title: feat: add patterns example app — Session 1 (scaffold + forms #1-7)
Suggested by: @Copilot
Comment type: review comment on patterns/templates/forms/bulk-update.tmpl:12

Task Description

The per-row "Active" checkbox inputs have no accessible name, so screen readers can't identify what each checkbox controls.

Proposed fix: Add an aria-label (e.g., including the user's name) or associate each checkbox with a <label>.

Original Comment

The per-row "Active" checkbox inputs have no accessible name. Consider adding an aria-label (e.g., including the user's name) or associating each checkbox with a <label> so screen readers can identify what each checkbox controls.

<td><input type="checkbox" name="active-{{.ID}}" aria-label="Active for {{.Name}}" {{if .Active}}checked{{end}}></td>

This issue was automatically created by prmonitor from PR review comments.

Metadata

Metadata

Assignees

No one assigned

    Labels

    follow-upFollow-up task from PR reviewfrom-reviewIssue originated from PR review

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions