Frontend Mentor | QR code component coding challenge solution

Solution retrospective

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

I am proud that i managed to do the challenge and try to make it look the same. Not sure how i would approach it differently next time but it's just the beginning

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

getting the content in the center but i got it at the end, just did some research

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

probably write cleaner code

Community feedback

Harsh Kumar Dwivedi• 740

@Harsh-Kumar-Dwivedi

Posted 6 months ago

Hi @Micha087 ,

I think you can make markup more semantic by using <main> element instead of <div class="card"> and <footer> element instead of <div class="attribution">. This will improve accessibility.

Hope it is helpful !

Happy Coding !

Micha087• 40

@Micha087

Posted 6 months ago

Hi @Harsh-Kumar-Dwivedi do you mean replace the divs with main and footer element or surround the <div class="card"> with the main element and <div class="attribution"> with the footer element?

Harsh Kumar Dwivedi• 740

@Harsh-Kumar-Dwivedi

Posted 6 months ago

@Micha087

Both will work, but it would be better to replace as it will not become extra code then and also you can style them using element selectors only instead of assigning classes.

Hope this helps !

Micha087• 40

@Micha087

Posted 6 months ago

@Harsh-Kumar-Dwivedi Thanks for the advice, i will keep it in mind for future challenges

Harsh Kumar Dwivedi• 740

@Harsh-Kumar-Dwivedi

Posted 6 months ago

@Micha087

Welcome !

If you find it helpful, please consider marking it as "Marked as helpful".

Happy Coding !

QR code component

Design comparison

Solution retrospective

Community feedback

Please log in to post a comment

Join our Discord community