Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
38
Comments
2
AlexCristea2007
@AlexCristea2007

All comments

  • Rahana Suleiman•190
    @Rahana23
    Submitted 8 months ago
    What are you most proud of, and what would you do differently next time?

    Am most proud of completing this project.

    I would make it more responsive.

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

    I encountered no challenges

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

    All suggestions are welcomed

    QR Code With HTML And CSS

    #pure-css
    1
    AlexCristea2007•840
    @AlexCristea2007
    Posted 8 months ago

    Hey Rahana, I came across your project I'd like to mention that your approach for this challenge was mostly perfect. The HTML structure is well built for CSS layout handling(small tip for semantic HTML: you can also rely on the <section> when you must use container elements and also try to use more specific classes than a bold "container" for more clarity. This'll be a great thing to keep in mind when dealing with larger projects). And for the CSS styles, you respected the UI requirements for the QR Card. The only minor flow that did a fairly noticeable change was the use of flexbox. The alignment should be vertical/faced as a column, so that's why you should include the "flex-direction:column" for the desired result. Hope this was helpful and good luck on your learning path, take small but precise steps!!👍

    Marked as helpful
  • feldspar58•350
    @feldspar58
    Submitted 12 months ago

    article-preview-component

    1
    AlexCristea2007•840
    @AlexCristea2007
    Posted 12 months ago

    The project is overall good, and I really want to leave a few suggestions in order to enhance some of its design and functionality. For starters, I have to emphasize a harmless but noticeable mistake, the .container doesn't contains in it's styles block any "box-shadow". Another detected issue is shown up inside the JS file due to displaying another button inside the dropdown in desktop design, instead there should be one. That's all bad news, apart from it you code looks good and fresh for this challenge, also I must admit that you HTML code doesn't break the web responsive' rule. Good job for your curageous attempt and I'm looking forward to seeing lots of improvements during your learning curriculum.

    Marked as helpful

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub