Skip to content

Restyle comment inputs with BB tokens - #69

Merged
brsbl merged 3 commits into
bb/hover-card-ready-captures-thr_bwikffmsvpfrom
bb/timeline-comments-bb-input-thr_bwikffmsvp
Aug 19, 2026
Merged

Restyle comment inputs with BB tokens#69
brsbl merged 3 commits into
bb/hover-card-ready-captures-thr_bwikffmsvpfrom
bb/timeline-comments-bb-input-thr_bwikffmsvp

Conversation

@brsbl

@brsbl brsbl commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keeps the existing Moss-like comment input structure and interactions while applying BB input, ring, background, recessed, muted, and destructive tokens.
  • Covers empty, focused, populated, submitting, validation-error, server-error, and disabled states without changing comment behavior.
  • Keeps the submit button exactly unchanged: same markup, icons, copy, dimensions, placement, hover/disabled appearance, keyboard shortcut, and submission behavior.

Visual evidence

Both revisions use the same real BB dev app, thread selection, fixture text, 1440x900 viewport, and window bounds.

Before — browser-native textarea focus After — BB focus treatment on the whole input
Before comment input focus After comment input focus
Empty + disabled Populated Submitting
Empty disabled comment input Populated comment input Submitting comment input
Validation error Server error with draft retained
Comment validation error Comment server error

Validation

  • npm run check --workspace=bb-plugin-timeline-comments — 7 files, 66 tests
  • npm run test:browser --workspace=bb-plugin-timeline-comments
  • Browser fixture verifies compact/multiline transitions, empty/disabled, focus, populated, validation error, submitting, failed-submit recovery, editing, portals, and responsive re-entry
  • git diff --check

BB-Thread-ID: thr_bwikffmsvp

@brsbl
brsbl marked this pull request as ready for review August 19, 2026 18:56
@brsbl
brsbl merged commit 77bad57 into main Aug 19, 2026
13 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