Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Day 6 of learning front-end (I think)

Glozsa 180

@Glozsa

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Nice 20 mins practice

Community feedback

Arlagonix 410

@arlagonix

Posted

Nice solution (I think)

Consider adding draggable="false" to the img so people wouldn't accidentally drag it around

Consider reducing space between header and paragraph. Closely related things must be close to each other. Presently the header is closer to the image than to the paragraph

The card isn't responsive when the screen width is less than 343px. Actually there are usually no screens with such small width, but still

Btw awesome shadow

Overall great solution

Marked as helpful

1

Glozsa 180

@Glozsa

Posted

@arlagonix I'll certainly try it out, especially the non draggable image thanks :D

0
Glozsa 180

@Glozsa

Posted

@arlagonix All fixed, thanks for your suggestions :D I don't think there's 250px screens, but to be fair, added that responsiveness aswell.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord