Skip to content
Submitted 4 days ago

HTML/CSS Flexbox + Figma Design

Michaeldremy•40
@Michaeldremy
Preview site (opens in new tab)View code (opens in new tab)
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?

Not much to write here

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

Learning how the style guide needs to be read to understand it. Learning spacing in the Figma design file.

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

I would like to know if targeting the body for background color was correct or should I have made a container and wrapped my HTML in that container.

How can I improve my class naming.

Is there a way to streamline my CSS? (I kept the styles in the style tag intentionally)

Community feedback

  • Marvel123g•80
    @Marvel123gPosted 4 days ago

    Your solution is perfect and your code is neat

    1
  • Adriano•39,410
    @AdrianoEscarabotePosted 2 days ago

    Hey Michaeldremy, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:

    Consider using rem for font sizes. When font sizes are set in absolute units like pixels, users can't adjust the text size based on their preferences. Relative units like rem adapt to the screen size and user settings, making them more flexible across various devices.

    If you'd rather keep using px, you can download a handy VS Code extension that converts pixels to rem automatically

    link -> px to rem Everything else looks great.

    Hope this helps! 👍

    0

Please log in to post a comment

Log in with GitHub

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