fix: update social card#149
Conversation
Deploying image-js-docs with
|
| Latest commit: |
0b5cff4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2dc3ab78.image-js-docs.pages.dev |
| Branch Preview URL: | https://146-change-open-graph-image.image-js-docs.pages.dev |
stropitek
left a comment
There was a problem hiding this comment.
I suppose we cannot test this unless we merge it.
For the social card, I would suggest you do an iteration in which you remove the top "ImageJS" text. It's repetitive (already in the logo) and does not look great. You can also try to find a font which fits well for the text on the right.
|
@EscapedGibbon Please temporarily change the website url so we can test here, and revert before merging. |
0836bb3 to
0f4ed1c
Compare
There was a problem hiding this comment.
LGTM
Need to revert the URL and decide which image we use (and delete the other one)
I think the work you started with the social card is good enough if just improved slightly. THe layout bothers me a little, the text on the right should be more balanced, have larger margins and more inter-line space.



No description provided.