Submitted 5 months ago

QR Code using HTML and CSS

Shiv• 350

@undrthegraveyard

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

QR code component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I am not satisfied with how it turned out, it could have been better and would have taken way less time.

Next time, I will definitely think more logically before writing my code in HTML and CSS.

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

Foremost challenge I encountered was to make the image-container look the same as given in the brief, I used ChatGPT to help me with it.

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

I would love some feedback on my structure of the code. My goal is to write as efficiently as possible.

Community feedback

Josh Pickard• 130

@joshpickardme

Posted 5 months ago

You say you are "not satisfied with how it turned out" yet you've produced something that is very close to the design.

Some very simple improvements that you could make to help you write as efficiently as possible.

Change the HTML element for the text "Improve your front-end skills by building projects" from a <p> to a <h1>. Remove the class attribute from both text elements as it's now no longer needed.
In styles.css you can select h1 and p instead of having selecting classes that are .p1 and .p2. It makes sense semantically that the bolder text is the heading.

Hope this helps

Marked as helpful

Shiv• 350

@undrthegraveyard

Posted 5 months ago

@joshpickardme

I really appreciate your feedback. I am going to take note of what you said, quite helpful indeed.

Thank you!

Please log in to post a comment

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