Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive Frontend Mentor - QR code component using flexbox.

John Carlo 170

@JohnCarloCanada

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any advices? do's and don't I should follow? Thanks in advanced.

Community feedback

@VCarames

Posted

Good job on your project!

One good practice is knowing when to use classes. Since this project is really small (code wise), classes are not necessary. This project uses only one of each element; main, img, heading and p.

Once your project start to get larger, then classes are definitely required. To keep things classes clean and easy to read you can use the BEM naming convention. Here's an article that explains more in detail. https://codeburst.io/understanding-css-bem-naming-convention-a8cca116d252

Also, each element serves a purpose. For example, the header element is usually reserved for introductory content; nav and hero content. https://developer.mozilla.org/en-US/docs/Web/HTML/Element

So for this project, you use either the main or article element.

Hope this helps in your coding journey!

Marked as helpful

0

John Carlo 170

@JohnCarloCanada

Posted

@vcarames Thanks sir I'll take note of this.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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