Skip to content

Mail: show labels for each email row #348

@elie222

Description

@elie222
Owner

We only need to load labels once for the entire screen. useLabels hook should handle this for you already.

Activity

added a commit that references this issue on Apr 7, 2025
f357b76
rainwaters11

rainwaters11 commented on Sep 25, 2025

@rainwaters11

Proposal: Add compact, accessible label chips to each email row in the Mail list.
Why: Improves scanability and triage without adding confusion or layout shift. Keeps list virtualization performance intact.
Acceptance Criteria:

Up to 3 labels shown per row with overflow as “+N”.

Labels inherit theme colors; maintain AA contrast.

Fully keyboard and screen-reader accessible (aria labels, no tab traps).

No regressions to list virtualization, line height, or LCP.

Works when a message has 0 labels, many labels, or very long label names.
Deliverables: Small PR with before/after screenshots, unit tests for rendering/overflow, and a short a11y note.
If this approach works, please assign me and I’ll open a PR.

rainwaters11

rainwaters11 commented on Sep 25, 2025

@rainwaters11

Proposal: Show compact, accessible label chips on each email row using the existing useLabels hook (labels loaded once per screen).

elie222

elie222 commented on Sep 29, 2025

@elie222
OwnerAuthor

Closing this issue as we no longer show the Mail page to users

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @elie222@rainwaters11

      Issue actions

        Mail: show labels for each email row · Issue #348 · elie222/inbox-zero