Skip to content
Submitted 8 months ago

Flex QR code component

Vishal Pawar•200
@vishalpawarR
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?

I learnt basics of figma designs and it was very helpful how the properties helped me build this project ASAP.

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

I was facing issue initially using the Figma but I was able to create a issue in help channel and was able to solve with people help.

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

I need few inputs regarding these:

  • I want help in checking I have added the heading thing in p tag should I use any specific tags for bold point?
  • I have not added any media query as the card size is very small that is 320px and do we need to add some margin for the below screen size of 375px?
  • Please review my class naming convention and html schema
  • How to check the font size in the Figma file? I tried searching it but it was not helpful

Community feedback

  • Sitt Shein•40
    @sittsheinPosted 8 months ago

    I installed Figma app and I was able to view the font size also. https://help.figma.com/hc/en-us/articles/360039956634-Explore-text-properties

    I hope this helpful! Above all, the solution you submitted is great!

    Cheers!

    Marked as helpful
    1
  • Adrian R.•50
    @adriarodrPosted 8 months ago

    Hello Vishal Pawar, For the card-heading, I recommend utilizing a heading tag (h1-h6) to ensure accessibility and the tag provides bold styling. I consider eliminating the qr-content div unless it serves a specific purpose. Other than that, everything looks great. Regarding the font size, I struggled at first to find the font size in the Figma file. To find the font size, I clicked on the background (not within any frame) to reveal the local style on the right panel. The fonts and font size will be underneath the text style. The heading is 22px, and the body is 15px.

    Marked as helpful
    1

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