Skip to content

Visor: make glyph selection a modal with Random action - #201

Merged
lannbot merged 1 commit into
mainfrom
visor/random-in-glyph-picker
Sep 11, 2026
Merged

Visor: make glyph selection a modal with Random action#201
lannbot merged 1 commit into
mainfrom
visor/random-in-glyph-picker

Conversation

@lannbot

@lannbot lannbot commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move glyph randomization into the picker as an always-active Random action; remove outer glyph dice, Clear glyph, and glyph-specific roll eligibility.
  • Open the picker as a native modal dialog with outside-click/tap, Escape, and Close dismissal; preserve selection focus restoration and Save/Revert.
  • Constrain the panel to the dynamic viewport with internally scrolling results.

Verification

  • Passed Rust/Deno checks and tests, Wasm build, and composed site build.
  • All 33 Chromium scenarios passed, including native modal state, keyboard focus containment, outside click/tap without background activation, Random/manual selection, and persistence.
  • Verified a 390x400 viewport retains a 12px gutter with Random, search, and Close visible.

Automerge will be armed after creation.

@lannbot
lannbot enabled auto-merge September 11, 2026 22:21
@lannbot
lannbot merged commit b5886e4 into main Sep 11, 2026
2 checks passed
@lannbot
lannbot deleted the visor/random-in-glyph-picker branch September 11, 2026 22:27
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.

2 participants