Skip to content
OverviewSolutions
2
Comments
3
Hugo F Hernandez
@HugoHern

All comments

  • Lagxy•110
    @LagxySubmitted almost 2 years ago

    the h2 font color doesn't changing on my view, why?

    just basic

    1
    Hugo F Hernandez•50
    @HugoHernPosted almost 2 years ago
    • Use ".text > h2 " in your CSS file instead to make sure you're selecting the correct element.
    0
  • AK•40
    @angelkendallSubmitted almost 2 years ago

    Hello, In this challenge I think I may have overdone it with the CSS - I reckon that it possibly could've been done in a much simpler way? So, I'm looking for any potential feedback on how to best potentially simplify CSS.

    I am also concerned that I am reinforcing bad HTML coding habits, so if you see any mistakes or bad habits please feel free to let me know.

    Thanks in advance

    QR Code Component (HTML/CSS)

    2
    Hugo F Hernandez•50
    @HugoHernPosted almost 2 years ago
    • Your solution is really close to the original
    • Your CSS looks great, its really clean and organized
    • The only criticism I have is the spacing between elements
    • It looks like you only need to tweak the font sizes in your html
    Marked as helpful
    1
  • Hugo F Hernandez•50
    @HugoHernPosted almost 2 years ago

    • Your solution is really close but I believe you just need to align your text to the center to get closer to the solution.

    • It looks like the corners need to be just a bit more rounded on the main-card.

    -You also need to add the font-family included in the design folder to get the same font-style.

    -Good job!

    0