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

@trueberrylesstrueberryless commented Jul 3, 2026

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 marginWith margin
imageimage
Production linkDeployment preview

@vercel

vercelBot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentJul 3, 2026 5:00pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewJul 3, 2026 5:00pm
npmx-lunariaIgnoredIgnoredJul 3, 2026 5:00pm

Request Review

@coderabbitai

coderabbitaiBot commented Jul 3, 2026

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 nameStatusExplanation
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
Title check✅ PassedThe title matches the main change: adding spacing to the Bluesky post embed engagement row.
Description check✅ PassedThe 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

codecovBot 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!

@ghostdevvghostdevv changed the title fix: bluesky post engagement marginfix: spacing on bluesky post embeds with a gallery on the blogJul 4, 2026
@ghostdevv
ghostdevv added this pull request to the merge queueJul 4, 2026
Merged via the queue into npmx-dev:main with commit e304adaJul 4, 2026
26 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Jul 4, 2026
@trueberryless
trueberryless deleted the fix/bluesky-post-engagement-margin branch July 4, 2026 06:23
ayo-run pushed a commit to ayo-run/npmx.dev that referenced this pull request Aug 5, 2026
Sign up for freeto 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.

2 participants

@trueberryless@ghostdevv