Submitted over 2 years ago

QR-Code-component-main

#accessibility

Kevin• 160

@Kevin-Kigotho

Preview site View code

This is a solution for...

QR code component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Please give any feedback, I will highly appreciate it.

Community feedback

Dolapo• 820

@nofeesahdee

Posted over 2 years ago

Great job Kevin, For the accessibility, instead of using div, try use semantic html5 such as main

Marked as helpful

Kevin• 160

@Kevin-Kigotho

Posted over 2 years ago

@nofeesahdee Thank you for your feedback. Let me apply that when I redo it.

Fluffy Kas• 7,735

@FluffyKas

Posted over 2 years ago

Hey,

Good job on this one, looks pretty close! Perhaps it would be a better idea to add the image to the HTML instead of using it as a background since this QR code pic happens to be the main content of this whole component :)

Marked as helpful

Kevin• 160

@Kevin-Kigotho

Posted over 2 years ago

@FluffyKas Thank you for the appreciation. Let me try that.

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