-
Notifications
You must be signed in to change notification settings - Fork 779
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
feat: og image for link (shared file) #778
base: main
Are you sure you want to change the base?
Conversation
@zaCKoZAck0 is attempting to deploy a commit to the mftsio Team on Vercel. A member of the Team first needs to authorize it. |
All contributors have signed the CLA ✍️ ✅ |
@zaCKoZAck0 looks good for now. Let me know if you have any questions |
I have read the CLA Document and I hereby sign the CLA |
@zaCKoZAck0 are you still working on this issue? |
Hi @mfts ,
Few workarounds:
What path should I take next?, I've updated the current state of OG preview in the PR description. |
@zaCKoZAck0 thanks for the transparency. Perhaps you can then design a OG image that takes the name of the document, Papermark and then a standard papermark image. Something that we could swap out in the future as a standard image. |
@zaCKoZAck0 let me know if you'd like to proceed with this PR? |
Hi @mfts , Here's from another office suite. We can replace the grey area of the og image with a placeholder illustration image like this. That will let the user know what type of document it is. |
For Sheet, Image, Slides, Dataroom and other files @mfts please review the above placeholder images. |
@zaCKoZAck0 these placeholders aren't really professional looking. I'm thinking we need to go basic and minimalist. |
Hi @mfts |
@mfts this is a more minimal option V3 |
@zaCKoZAck0 I like this one this one the best (without the avatar) |
Thanks, I'll do the required changes. |
Hi @mfts It's ready for review now. |
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.
Looks super cool!
Thanks for working with me to get this over the line! May I ask where you go the tiled background from. I may want to reuse it in other places :) |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hi @mfts Here's the background I used https://www.svgbackgrounds.com/set/free-svg-backgrounds-and-patterns/#pattern-randomized Use stroke width around 0.1 0.2 to get that github like grid 😁 |
/award 1500 awarding you and merging it later |
Awarding zaCKoZAck0: 1500 points 🕹️ Well done! Check out your new contribution on oss.gg/zaCKoZAck0 |
Related to #629
[WIP]: add og images for file sharing url.
Draft:
Currently og image looks something like this:
Last updated 30-10-24