(Fixed) Documentation: Update Textmagic and Sendgrid screenshots #1173
Conversation
There was a problem hiding this comment.
Thanks for the PR! 🙏🏼 The screenshots look a little small and don't seem to have an aspect ratio that matches our style guide. Would you please make sure:
- Screenshots must be 16:9
- Screenshots should be taken in a 1400 x 900 view port on 3x DPR in browser developer tools.
|
@stnguyen90 sure, I'll fix that |
There was a problem hiding this comment.
The screenshot doesn't look cut off anymore, but the DPR doesn't seem right. I expect the image to be much larger with a larger DPR.
One of the reasons why we need a higher DPR is to maintain clarity. When comparing the two images, the existing image looks much clearer. See the comparison between the new screenshot on the left and the existing image on the right:
|
@stnguyen90 but I'm pretty sure I set the DPR to 3 this time. I rechecked it after each screenshot to verify the same. Should I change the DPR from 3 to a higher value? |
Fixes #1088
What does this PR do?
Updated the screenshots at https://appwrite.io/docs/products/messaging (Providers) with updated logos of Textmagic and Sendgrid.
Test Plan
Manually Tested.
Related PRs and Issues
#1088
Have you read the Contributing Guidelines on issues?
Yes