Skip to content

Grinder SWAP and pick buttons: Enter opens the station; no machine names for screen readers #416

Open

Description

@jackgary86-dev

grinderBox stops click propagation but not keydown; the column's keydown calls preventDefault() and opens the station, so a keyboard user cannot swap. Add box.addEventListener('keydown', e => e.stopPropagation()) as the settings box does.

Activity

  1. jackgary86-dev commented on Oct 16, 2026

    @jackgary86-dev
    OwnerAuthor

    Also (accessibility): the new pick and SWAP buttons read only 'BUY & PLACE $3,500' / 'PLACE · in the yard' / 'SWAP · $14,000' with no aria-label naming the machine; add e.g. 'Buy and place Sink-float tank, $3,500'. The ONE STEP AT A TIME / WHOLE PLANT group sits inside the Plant h2, so the heading's accessible name runs them together; move it out of the heading.

  2. changed the title [-]Enter on a grinder SWAP button opens the station instead[/-] [+]Grinder SWAP and pick buttons: Enter opens the station; no machine names for screen readers[/+] on Oct 16, 2026
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

    bugSomething isn't workinggui

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions