-
Notifications
You must be signed in to change notification settings - Fork 4.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: og preview #2633
fix: og preview #2633
Conversation
✅ Deploy Preview for vuejs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Logo as an og:image is not a good idea. It will be stretched/cropped most likely while sharing. 1200 x 630 px image is usually recommended. |
I just found that vue2 is using the same image. Is there any better image to use? |
Yeah but in the previews you see how that thing is looking weird. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adjusted things a bit. This looks fine for most of the stuff except LinkedIn (and certain padding issues if one looks closely). I'll ask Kia to design some cool image for this and we can replace that later.
https://www.opengraph.xyz/url/https%3A%2F%2Fdeploy-preview-2633--vuejs.netlify.app%2F
Fixes: #2631