close
Skip to content

[2.x] fix: add left padding to posts on mobile#4422

Merged
imorland merged 1 commit into2.xfrom
im/fix-mobile-post-padding
Mar 8, 2026
Merged

[2.x] fix: add left padding to posts on mobile#4422
imorland merged 1 commit into2.xfrom
im/fix-mobile-post-padding

Conversation

@imorland
Copy link
Copy Markdown
Member

@imorland imorland commented Mar 8, 2026

Summary

  • Fixes uneven post padding on mobile devices (issue [2.x] Mobile Post Formatting #4415)
  • .Post base style has padding-left: 0 since the avatar grid column provides implicit left spacing on desktop/tablet; on mobile the grid and avatar are removed, leaving content flush against the left edge
  • Adds padding-left: var(--post-padding) inside @media @phone so both sides have symmetrical 20px padding

Test plan

  • Open a discussion on a mobile device or browser devtools at ≤767px viewport width
  • Verify posts have equal left and right padding on both comment posts and event posts
  • Verify event post icons still align correctly
  • Verify thread divider lines between posts still render correctly

Closes #4415

🤖 Generated with Claude Code

On mobile, the avatar column (which provides implicit left spacing on desktop) is hidden and the grid layout removed. The `.Post` base style has `padding-left: 0`, causing post content to be flush with the left edge while having 20px on the right — visually uneven. Add `padding-left: var(--post-padding)` under `@media @phone` to restore symmetrical spacing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@imorland imorland requested a review from a team as a code owner March 8, 2026 16:53
@imorland imorland changed the title fix: add left padding to posts on mobile [2.x] fix: add left padding to posts on mobile Mar 8, 2026
@imorland imorland added this to the 2.0.0-beta.8 milestone Mar 8, 2026
@imorland imorland merged commit 60a6920 into 2.x Mar 8, 2026
27 checks passed
@imorland imorland deleted the im/fix-mobile-post-padding branch March 8, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[2.x] Mobile Post Formatting

1 participant