-
Notifications
You must be signed in to change notification settings - Fork 0
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
Snapshot of Page does not include image #264
Comments
According to this Facebook official article:
The dimensions of this snapshot screenshot above are 840 × 550 I think we are not using the Open Graph Stories Images, so we have to modify the screenshot image ratio to 🎅🏻🎅🏻🎅🏻 |
Ok, sounds good, so if we are going to use I can make all the changes! I am in Mexico for another week! But we should hang out once I am back! |
@ooookai I've been experimenting with the image ratios a little bit, but it seems that the problem might be how we are embedding the pic in the document. For example, embedding this snapshot does not look like this one in the DOM. I think the way the DOM is constructed makes FB to not fetch it |
@ooookai although it works with Twitter |
@ooookai I just doubled checked. You are right it is a problem of the dimensions. I will put together a fix. |
@ooookai I made the changes so the images are now 1.91x1.0. If I try to embed the image from the PWD link, it still doesn't work. But when I upload manually to S3 instead, it does work. Also when I paste the actual image from S3, it does work Perhaps is an issue with the CSS or HTML in our page? Or maybe it is the way it is being uploaded to S3? @ooookai What do you think? |
After some study... I knew this problem, So the one I used for the snapshot page was copied from the 'medium.com' website and modified by myself, which should work perfectly.😰 However, you may give these two things a quick try.
<meta property="og:image:type" content="image/jpeg" />
You could use this tool from Facebook to test if it works. |
I don't know if we can just send a quick bug report to Facebook. |
@ooookai Let me give it a try, push to the server, etc! Sounds good! Thanks for checking it out! |
@ooookai where is |
@cesandoval Just saw your question! 😉
The file path: |
@ooookai but which line of code, there are a lot of |
The page preview should also include the image snapshot.
Right now it only includes text
The text was updated successfully, but these errors were encountered: