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 using css flex, html

Felista5β€’ 20

@Felista5

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


some areas like the css box model was difficult for me

Community feedback

Panjiβ€’ 2,110

@pperdana

Posted

Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

  • I have some additional recommendations for your code that I think you'll find interesting and valuable.

πŸ“Œ Add <main> tag as semantic HTML in code

  • The <main> tag is a semantic HTML element that is used to define the main content of a web page.

  • The <main> tag should be used to wrap the primary content of a web page, such as the main article, section, or body of text.

for example code:

<main>
  <div class='container'>
    <h1>Article Title</h1>
    <p>Article content goes here...</p>
    .......................
  </div>
</main>

In the example above, the <main> tag is used to wrap the <div> tag, which contains the primary content of the web page. This tells both human readers and search engines that the content inside the <main> tag is the most important and relevant content on the page.

I hope you found this helpful!

Happy codingπŸ€–

Marked as helpful

1

Felista5β€’ 20

@Felista5

Posted

@Panji200 Noted. Thanks

0
Ecem Gokdoganβ€’ 9,380

@ecemgo

Posted

Some recommendations regarding your code that could be of interest to you.

  • If you use max-width instead of width, the card will be responsive
#container {
  /* width: 300px; */
  max-width: 300px;
}
  • You've defined font-family and font-weight and then you've defined the same values in texts again. I recommend you try to avoid repetition.
#first {
  /* font-family: "Outfit", sans-serif; */
}
#second {
  /* font-weight: 400; */
  /* font-family: "Outfit", sans-serif; */
}
  • Finally, you'd better remove width from the text. After updating it, you won't have view like in screenshot and it will look better
p {
  /* font-size: 15px; */
  /* margin: auto; */
  /* width: 90%; */
}

Hope I am helpful. :)

Marked as helpful

1

Felista5β€’ 20

@Felista5

Posted

@ecemgo Thank you very much, I have taken note of them.

1
Ecem Gokdoganβ€’ 9,380

@ecemgo

Posted

@Felista5 happy to help! :)

1
jordanBuilderβ€’ 60

@jordanBuilder

Posted

hey ,what's up ?.. have you done it by yourself ..?

1

Felista5β€’ 20

@Felista5

Posted

@jordanBuilder I'm learning more about it by building projects. Thank you.

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