Submitted 2 months ago

QR Code Component using Flex

Soutrik Das•30

@SoutrikDas

Preview site View code

This is a solution for...QR code component

HTML
CSS

1newbie

Design comparison

SolutionDesign

Solution retrospective

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

Proud of : It looks great , even though I didnt design any of it ( colors , fonts and spacing ) , it still looks good

Do Differently : I think since I made the qr class name and gave it to the div , that I should have used its margin property instead of the img , maybe ? I think I have more divs than I actually need.

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

Initially I was using this to make distance between the image and the text :

img {
  height: 100%;
  width: 100%;
  border-radius: 10px;
  padding-bottom: 24px;
}

This was causing the Image to have rounded top corners but normal sharp bottom corners. I was confused at first but later figured out and changed it to margin-bottom which solved the problem

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

Structure of the html file , I think I used too many divs and maybe not even properly used them.

Community feedback

MikDra1•6,980
@MikDra1Posted 2 months ago
If you want to make your card responsive with ease you can use this technique:

.card { width: 90%; max-width: 37.5rem; }

On the smaller screens card will be 90% of the parent (here body), but as soon as the card will be 37.5rem (600px) it will lock with this size.

Also to put the card in the center I advise you to use this code snippet:

.container { display: grid; place-items: center; }

Hope you found this comment helpful 💗💗💗

Good job and keep going 😁😊😉
Marked as helpful
1
Steven Stroud•11,910
@StroudyPosted 2 months ago
Awesome job tackling this challenge! You’re doing amazing, and I wanted to share a couple of suggestions that might help refine your approach…

Using a <main> tag inside the <body> of your HTML is a best practice because it clearly identifies the main content of your page. This helps with accessibility and improves how search engines understand your content.

These <div> should really have semantic tags like headings (<h1> to <h6>) and paragraphs (<p>) convey structure and meaning to content, improving accessibility, SEO, and readability by helping search engines and screen readers interpret the content.

<div class="text-1 outfit-bold"> Improve your front-end skills by building projects </div>

Developers should avoid using pixels (px) because they are a fixed size and don't scale well on different devices. Instead, use rem or em, which are relative units that adjust based on user settings, making your design more flexible, responsive, and accessible. For more information check out this, Why font-size must NEVER be in pixels or this video by Kevin Powell CSS em and rem explained.- Another great resource for px to rem converter.

Using max-width: 100% or min-width: 100% is more responsive than just width: 100% because they allow elements to adjust better to different screen sizes. To learn more, check out this article: responsive-meaning.

Line height is usually unitless to scale proportionally with the font size, keeping text readable across different devices. Best practice is to use a unitless value like 1.5 for flexibility. Avoid using fixed units like px or %, as they don't adapt well to changes in font size or layout.

You’re doing fantastic! I hope these tips help you as you continue your coding journey. Stay curious and keep experimenting—every challenge is an opportunity to learn. Have fun, and keep coding with confidence! 🌟
Marked as helpful
1

Please log in to post a comment

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