Skip to content

fix(thread): reduce excessive whitespace between root message and replies#302

Merged
Just-Insane merged 3 commits intodevfrom
fix/thread-empty-space
Mar 16, 2026
Merged

fix(thread): reduce excessive whitespace between root message and replies#302
Just-Insane merged 3 commits intodevfrom
fix/thread-empty-space

Conversation

@Just-Insane
Copy link
Collaborator

Summary

Fixes #294 — There was approximately 84px of dead whitespace between the thread root message and the first reply in the thread drawer.

Root cause

  • Root message box had padding: S600 0 (adding S600 ≈ 32px at bottom)
  • Reply messages box had padding: S600 0 (adding S600 ≈ 32px at top)
  • The reply count label between them added its own S200 spacing

Fix

  • Root message box: padding: S600 0padding: S400 0 S200 0 (reduced bottom padding; the label provides visual separation)
  • Reply list box: padding: S600 0padding: 0 0 S600 0 (removed top padding; label + root message bottom padding is sufficient)

Net result: ~84px of dead space reduced to ~32px.

The thread drawer had padding: S600 0 (24px top+bottom) on both the root
message box and the reply messages box, plus S200 padding on the reply count
label. This stacked up to ~84px of empty space between the root message and
the first reply, especially noticeable in newly created threads.

Fix:
- Root message box: change bottom padding S600→S200 (top kept at S400)
- Reply messages box: remove top padding (reply count label already provides
  visual separation)

Fixes #294
@Just-Insane Just-Insane requested a review from a team March 16, 2026 19:08
@github-actions
Copy link
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

Status Preview URL Commit Alias Updated (UTC)
✅ Deployment successful! https://pr-302-sable.raspy-dream-bb1d.workers.dev 1124ba1 pr-302 Mon, 16 Mar 2026 19:37:28 GMT

@Just-Insane Just-Insane added this pull request to the merge queue Mar 16, 2026
Merged via the queue into dev with commit 135dbe9 Mar 16, 2026
8 checks passed
@Just-Insane Just-Insane deleted the fix/thread-empty-space branch March 17, 2026 03:43
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.

Too much empty space in a new thread

1 participant