Skip to content

FOLLOW-UP: Add aria-label to edit-row name/email inputs (from PR #59) #63

@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/edit-row.tmpl:15

Task Description

The inline row editor inputs have no associated <label> or aria-label/aria-labelledby, which makes the row editor difficult to use with screen readers.

Proposed fix: Add explicit aria-label attributes for the name/email fields (or use visually-hidden labels).

Original Comment

These inputs have no associated <label> (or aria-label/aria-labelledby), which makes the inline row editor difficult to use with screen readers. Add labels (they can be visually hidden) or add explicit aria-label attributes for the name/email fields.

<input name="name" value="{{.Name}}" aria-label="Name">
<input name="email" value="{{.Email}}" aria-label="Email">

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