Skip to content
OverviewSolutions
4
Comments
4
Carmen
@KrmenDev

All comments

  • Carmen•80
    @KrmenDevPosted about 2 months ago

    I was not able to see the code, but the solution looks pretty close to the design, everything looks clean, the only thing I can point out is the font size looks quite small maybe for the mobile version can stay that way but for desktop can be a little bit bigger

    Marked as helpful
    0
  • Carmen•80
    @KrmenDevPosted 3 months ago

    The result looks exactly like the design, everything looks great my only observation would be for font size it could be better to use rems instead of px that ways is more accessible for all users, also your CSS looks really clean, your class names are really descriptive so they are easy to read and understand what they are selecting, congrats

    Marked as helpful
    1
  • Carmen•80
    @KrmenDevPosted 3 months ago

    I think the solution looks great it looks like the design, I like how you solve the box shadow, and how you use the border to finish the design that was smart, everything looks good on a smaller screen and is easy to understand I think everything is also semantically correct

    1
  • Rashid•100
    @Rashid0101Submitted 3 months ago
    What are you most proud of, and what would you do differently next time?

    Hi there 👋, I’m Rashid and this is my solution for this challenge. 🚀

    Any suggestions on how I can improve and reduce unnecessary code are welcome!

    Thank you. 😊

    QR-code-component

    2
    Carmen•80
    @KrmenDevPosted 3 months ago

    I think It looks great, you achieve the result almost identical to the design

    0