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 Card Component

Fatihβ€’ 30

@mFatihGorhan

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


This difficulty was easy for me. But if I still have any shortcomings, please let me know.

Community feedback

Ferβ€’ 3,970

@fernandolapaz

Posted

Hi πŸ‘‹, perhaps some of this may interest you:

HTML 🧱, ACCESSIBILITY βš–:

  • Every page should have an <h1> to improve user experience and because it is an important element when it comes to SEO.

CSS 🎨:

  • The page content could be centered using Grid or Flexbox. For example as follows:
body {
min-height: 100vh;
display: grid;
place-content: center;
}
  • Length units such as pixels may not be the best alternative because absolute units don’t scale. Relative units like rem or em are a better option for scalable layouts (the page will adjust to the user's browser settings) and maintenance (to make changes without having to adjust every pixel value).

Please let me know if you want more info on any of these topics or disagree with something. I hope it’s useful πŸ™‚

Regards,

1

Richard Kpehorβ€’ 100

@Richardedem6

Posted

@fernandolapaz that is awesome, I look forward to learning more.

0
Fatihβ€’ 30

@mFatihGorhan

Posted

@fernandolapaz Thank you very much for your constructive criticism. The information you provided is very valuable to me. However, I couldn't fully grasp the concept of rem and em and how to use them effectively. I am still struggling to understand how to apply them in practice. Also, I appreciate your guidance on using the grid for centering content.

0
Ferβ€’ 3,970

@fernandolapaz

Posted

@mFatihGorhan You are welcome.

Regarding the length units, you could easily start converting to rem (1 rem equals the font size of the root element, 16px by default).

Marked as helpful

0
Anuoluwapo-devβ€’ 80

@Anuoluwapo-dev

Posted

You have done a good work on this project , Keep on and continue improving your skills. but you have done good generally!!

0

Richard Kpehorβ€’ 100

@Richardedem6

Posted

@Anuoluwapo-dev thank you. I really appreciate the comment.

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