Skip to content

fix: spacing on bluesky post embeds with a gallery on the blog#2991

Merged
ghostdevv merged 2 commits into
npmx-dev:mainfrom
trueberryless:fix/bluesky-post-engagement-margin
Jul 4, 2026
Merged

fix: spacing on bluesky post embeds with a gallery on the blog#2991
ghostdevv merged 2 commits into
npmx-dev:mainfrom
trueberryless:fix/bluesky-post-engagement-margin

Conversation

@trueberryless

@trueberryless trueberryless commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

🔗 Linked issue

🧭 Context

Rendering BlueSky posts.

📚 Description

After reading the awesome recent blog post, I noticed some missing margin between the engagement section of a BlueSky post and images/videos.

This PR fixes the missing margin by adding mt-3 to the engagement section, here is a visual comparison:

Without margin With margin
image image
Production link Deployment preview

@vercel

vercel Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jul 3, 2026 5:00pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Jul 3, 2026 5:00pm
npmx-lunaria Ignored Ignored Jul 3, 2026 5:00pm

Request Review

@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This change adds a single Tailwind mt-3 utility class to the engagement-row <div> in the Bluesky post embed component, adjusting top spacing above the timestamp and like/repost/reply counts. No logic, data flow, or exported entities were modified.

Changes

Bluesky Post Embed Styling

Layer / File(s) Summary
Engagement row spacing adjustment
app/components/global/BlueskyPostEmbed.client.vue		 Added mt-3 class to the <div> wrapping DateTime and like/repost/reply counts to adjust top margin.

Compact metadata: +1/-1 lines, single file, low review effort.

Related issues: None specified.

Related PRs: None specified.

Suggested labels: style, low-risk

Suggested reviewers: None specified.

Poem:
A whisker of space, a class so small,
mt-3 nudges counts down the wall,
No logic bent, no data astray,
Just tidy pixels, come out and play. 🐰✨

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title matches the main change: adding spacing to the Bluesky post embed engagement row.
Description check ✅ Passed The description is directly about the same margin fix and mentions the mt-3 change.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@ghostdevv ghostdevv changed the title fix: bluesky post engagement margin fix: spacing on bluesky post embeds with a gallery on the blog Jul 4, 2026
@ghostdevv ghostdevv added this pull request to the merge queue Jul 4, 2026
Merged via the queue into npmx-dev:main with commit e304ada Jul 4, 2026
26 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 4, 2026
Open
@trueberryless trueberryless deleted the fix/bluesky-post-engagement-margin branch July 4, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ghostdevv ghostdevv ghostdevv approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@trueberryless @ghostdevv