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

made with css flexbox

@Benaks

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


Project was easy, but there could be issues. point em out please...

Community feedback

@luztherose

Posted

Hi Ben,

Congratulation on completing this challenge. Your solution looks great. I have some suggestions regarding your solution if you don’t mind:

  1. When writing your HTML always try to use semantics elements whenever possible. For example header, main, footer, etc.

  2. The text for the alt="code" attribute tries to make it as descriptive as possible since this is the text that will be displayed if the image cannot be loaded.

  3. Since you are using an external stylesheet which is excellent, you could move the style that is written internally in the HTML for the footer to your external CSS and remove it from your HTML.

Overall Excellent work! Hopefully, this feedback helps.

Marked as helpful

1
Damian 150

@damiandev22

Posted

Hello and congrats, you did well on this challenge. I think that In order to achieve a most acurate result you could measure the card in the given design, so you can have an idea of the size the card should have in your solution. for that you can use Photoshop because doing so blindy is more difficult. in this challenge the card has 320px wide. Only that for now. keep up.

Marked as helpful

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