Skip to content

Refine encounter builder workflow and shared UI system#158

Merged
Daren9m merged 2 commits into
mainfrom
codex/refine-encounter-builder-ui
Jul 20, 2026
Merged

Refine encounter builder workflow and shared UI system#158
Daren9m merged 2 commits into
mainfrom
codex/refine-encounter-builder-ui

Conversation

@Daren9m

@Daren9m Daren9m commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • restructure the encounter builder into a clear setup → review → act workflow
  • separate calculated difficulty from actions so status no longer reads as a button
  • move recipes, filters, map controls, exports, and print options into progressive disclosure
  • place monster review before forecasting or live combat
  • add shared semantic tokens, responsive workflow patterns, and a reusable UI style guide
  • improve labels, focus restoration, touch targets, form submission, and print contrast

Why

The encounter builder had accumulated unrelated controls in the same surfaces and action rows. Primary actions, utilities, export formats, and state indicators carried similar visual weight, making the intended flow difficult to understand. The difficulty indicator between Run Battle and Edit Details was a particularly confusing example.

Impact

DMs now get a more deliberate encounter-building sequence with one clear primary action per decision surface. Shared buttons, headers, filters, status treatments, semantic tokens, and documented design rules also provide a consistent foundation for the rest of the toolkit.

Validation

  • npm run typecheck
  • npm run lint
  • npm test — 670 tests passed
  • npm run build — 16 static routes generated

@Daren9m
Daren9m marked this pull request as ready for review July 20, 2026 07:31
@Daren9m
Daren9m merged commit 01156da into main Jul 20, 2026
1 check passed
@Daren9m
Daren9m deleted the codex/refine-encounter-builder-ui branch July 20, 2026 07:36
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