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

QR Code Component solution using basic HTML and CSS

@Rodrigo-Policarpo

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


What are you most proud of, and what would you do differently next time?

I'm most proud of, by the first time, I engaged myself on coding and finished my first code 100% by myself.

What challenges did you encounter, and how did you overcome them?

The mainly challenge was something basic: Fit the footer on page bottom. The solution shown up when I read an article about elements positioning using CSS.

What specific areas of your project would you like help with?

I'll be really grateful if someone could give some tips about things that i can improve on my code, pointing my mistakes and weak points on code writing.

Community feedback

Huy Phan 1,860

@huyphan2210

Posted

Hi, @Rodrigo-Policarpo

You did a great job with the challenge! I have a couple of suggestions for improvement:

  • Using height: 100vh on the body works for this specific challenge, but I'd recommend switching to min-height: 100vh instead. This ensures the body can expand if the content grows beyond the viewport height, preventing potential overflow issues and making it more flexible across different devices and content lengths.
  • I noticed you're already using a footer element, which is great! Why not replace the .card's div with a <main> element instead? Since <main> semantically represents the main content of a page, it could be a better fit than a generic div, and it would make your HTML more accessible and meaningful.

Hope this helps!

Marked as helpful

0

@Rodrigo-Policarpo

Posted

Hi @huyphan2210 !

First of all, thanks for sharing these suggestions and code improvements!

  • The suggestion about the height tag was very useful, since I don't have much practice on ​​how to use min-max correctly.

  • Also, the suggestion to replace the elements was very important too.

Thank you for everything!

1

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