Skip to content
Submitted 6 months ago

Encore

Emmanuel Moolongawi•110
@EMLzmile
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?

Sorry

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

Sorry

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

Sorry

Community feedback

  • Lawrence Lee•70
    @LawrenceLCodesPosted 6 months ago

    Good project!

    HTML is fairly clean and CSS is well organized. I also approve of the use of CSS variables, and they help make the page easier to update as it expands in scope.

    The only areas of improvement are to clean up both files by removing the defaults/starter markup on the files. Finally, if you have the time, removing the default text from the README would help with the presentation of the repository on GitHub.

    Marked as helpful
    1
    Emmanuel Moolongawi•110
    @EMLzmilePosted 6 months ago

    Thanks for this advice, I will try to improve it the readme still bothers me, I have a hard time getting used to it. I really appreciate your opinion @LawrenceLCodes

    1
    Lawrence Lee•70
    @LawrenceLCodesPosted 6 months ago

    @EMLzmile

    It took me awhile to get the README file to look right on my end, so the feeling is understandable.

    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