Skip to content

Connect a computer: pick scripts with a row of toggle buttons - #197

Merged
Maselkov merged 1 commit into
mainfrom
connect-script-buttons
Nov 4, 2026
Merged

Maselkov merged 1 commit into
mainfrom
connect-script-buttons

Conversation

@Maselkov

@Maselkov Maselkov commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Follow-up to #196. The script checkboxes, each with a line saying which Adapter it needs, made Connect a computer about 100px taller, so admins had to scroll to reach the command.

What changes

  • The picker is now one row: Install: [Crafts] [Power] [Network]. These are toggle buttons with the range buttons' highlighted look and aria-pressed. Each Adapter hint moved into its button's tooltip. The installer still warns when a script's Adapter isn't connected.
  • The picker still remembers the choice in the browser, and with nothing selected Copy stays disabled and the "pick at least one" note shows.
  • The docs (docs/in-game.md, README step 4 and docs/CONTEXT.md) say "buttons" instead of "tick".

Testing

pytest, node --test, e2e (it now also checks aria-pressed and the active class), node --check, the Lua tests, ruff, luacheck and shellcheck all pass. I also checked the section in dev.py in dark mode.

🤖 Generated with Claude Code

The three checkboxes, each with a hint line, pushed the command about
100px down. They're now one row of toggle buttons (aria-pressed, the
range buttons' active look) with each hint as the button's tooltip; the
installer still warns about a missing Adapter.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Maselkov
Maselkov merged commit dd6fa89 into main Nov 4, 2026
8 checks passed
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.

1 participant