Skip to content

fix: open graph image on reddit#1076

Closed
NandkishorJadoun wants to merge 1 commit intonpmx-dev:mainfrom
NandkishorJadoun:fix/og-image-dimensions
Closed

fix: open graph image on reddit#1076
NandkishorJadoun wants to merge 1 commit intonpmx-dev:mainfrom
NandkishorJadoun:fix/og-image-dimensions

Conversation

@NandkishorJadoun
Copy link
Copy Markdown
Contributor

will add more details later

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 6, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 6, 2026 7:12am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 6, 2026 7:12am
npmx-lunaria Ignored Ignored Feb 6, 2026 7:12am

Request Review

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 6, 2026

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
Copy link
Copy Markdown
Contributor

will take a look at this after the vacation/recharging period 👀

@ghostdevv ghostdevv self-requested a review February 15, 2026 02:30
Comment thread nuxt.config.ts
defaults: {
component: 'Default',
width: 1200,
height: 630,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
height: 630,
height: 627,

There is some research that says this is the most correct here: cloudinary-community/svelte-cloudinary#159 (comment)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@harlan-zw you're the author of the og image lib right? wdyt?

@harlan-zw
Copy link
Copy Markdown
Contributor

harlan-zw commented Mar 10, 2026

Nearly every platform renders perfectly fine with 2:1 these days and is the default for Nuxt OG Image currently.

image

I'd recommend sticking to that unless you see something dramatically bad.

@alexdln
Copy link
Copy Markdown
Member

alexdln commented Mar 18, 2026

We're also intentionally leaving plenty of space on the sides, so we don't have any critical reasons to minor changes to it for a specific platform. I'll close the PR, please reopen it if you want to continue working on it

@alexdln alexdln closed this Mar 18, 2026
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.

4 participants