Skip to content
Submitted 10 months ago

QR code component using HTML + CSS

Aldrin Sean Pereira•90
@AldrinSeanPereira
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

I have completed freeCodeCamps Responsive Web Design certification and was told to make projects using this platform

When building the project I found it most difficult in 2 areas:

  • where to code the HTML and CSS
  • understanding the README and how to start putting code on Github

How it was fixed: Thanks to Alex for his amazing help, I was able to fix these. I had never used VS Code or knew how to use LiveServer and had to search for how to use all these

Suggestions:

  • It would be really helpful to include resources on how to code locally (since FCC uses an online editor) in the README and the discord challenge roadmaps
  • The README was overwhelming at first even though it is so well written then I realised I had to use "preview" in VS code to see it more easily. I only found this by accident but it was so helpful

Any feedback on my code is really appreciated. Thank you very much for everything!!

Community feedback

  • Daniel 🛸•44,250
    @danielmrz-devPosted 10 months ago

    Hello @AldrinSeanPereira

    Your solution looks great!

    I have a suggestion for improvement:

    • For semantic reasons, and since that is the main title of the screen, you can replace the <p> with <h1>.

    The <h1> to <h6> tags are used to define HTML headings. <h1> defines the most important heading. <h6> defines the least important heading. Only use one <h1> per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>, then use <h2>, and so on.

    I hope it helps!

    Other than that, you did a great job!

    1
    Aldrin Sean Pereira•90
    @AldrinSeanPereiraPosted 10 months ago

    @danielmrz-dev Thank you so much! Your advice definitely helps me improve my code.

    0
  • Marcos Travaglini•4,840
    @BlackpachamamePosted 10 months ago

    Hey! Your solution looks great, the truth is that you have handled it very well.

    I just have a few details to mention that may help you:

    • The favicon image is not visible, this is because you placed the path wrong, you must replace href="./images/favicon-32x32.png" with href="./favicon-32x32.png"
    • The <div class="attribution"> could be <footer class="attribution">
    1
    Aldrin Sean Pereira•90
    @AldrinSeanPereiraPosted 10 months ago

    @Blackpachamame Thank you so much!

    I will make these changes in my code. Do you know what the "./" means?

    0
    Marcos Travaglini•4,840
    @BlackpachamamePosted 10 months ago

    @AldrinSeanPereira

    • / = Root directory
    • . = This location
    • .. = Up a directory
    • ./ = Current directory
    • ../ = Parent of current directory
    • ../../ = Two directories backwards

    But, it is recommended that for github pages you always use the ./ in front of the path, because for some reason it usually generates errors.

    More info:

    2
    Aldrin Sean Pereira•90
    @AldrinSeanPereiraPosted 10 months ago

    @Blackpachamame Wonderful. Thank you soooooo much!

    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