women-who-software / scrollodex

A fun place for women who code members to add and display their business cards.

https://scrollodex.netlify.app/

MIT License

7 stars 47 forks source link

Updating card styling #148

Closed rweisberger closed 2 years ago

rweisberger commented 2 years ago

Summary of changes

Related Issue

How to test (for features) / reproduce (for bugs)

Screenshots

Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation (explains but does not affect functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Notes

please consider including a gif that reflects your feelings about this PR

netlify[bot] commented 2 years ago

Deploy Preview for scrollodex ready!

Name	Link
Latest commit	c0bc371d31c3fc06013ed187e2a1245645b3473e
Latest deploy log	https://app.netlify.com/sites/scrollodex/deploys/63597ac17fee4700085d1934
Deploy Preview	https://deploy-preview-148--scrollodex.netlify.app/
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

domsbytes commented 2 years ago

It looks great @rweisberger ! the background image is just a little big and resource-heavy for your site to be rendered fast. I think the biggest our cards will display background image is around 800px, can you resize it to be 800px wide please? If you have a mac you can do so in Preview then go to Tools -> adjust Size.