Submitted over 2 years ago

Responsive QR Code Component using grid

Danny• 10

@Danny-Rodriguez

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

How effective is my solution to centering each element?

Community feedback

Manuel Gil• 340

@ManuGil22

Posted over 2 years ago

Hey @Danny-Rodriguez pretty cool solution! Have only one thing to say as feedback:

Always use heading tags in order. Dont use <h4> as a first tag, you can use a <h1> instead and change the text-style with css!

Marked as helpful

Danny• 10

@Danny-Rodriguez

Posted over 2 years ago

@ManuGil22 Thanks, I figured my way of doing that was hacky

Prince Roy• 1,210

@iprinceroyy

Posted over 2 years ago

Please use the main element to wrap all the contents of a page & wrap class attribution elements inside the footer element. This way you can avoid html accessibility issue. Adjust padding/margin for mobile view as contents of card are collapsing. Happy coding :)

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