Plain HTML and CSS Grid

Solution retrospective
Comments/sugestions are more than welcome!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @SzymonRojek
Hi Nicholas,
Well done :D
I have checked your project, mainly HTML, a few tips from me:
- I think you don't need this div class="pricing";
- instead of two spans you can create two paragraphs or one paragraph and inside of it one span;
- on a page you can have only one h1 (please read about it why this is matter) so in the third box change h1 to h2;
- instead of button I suggest to use a link;
- remove CSS styles and transfer them to the CSS file;
- you don't have to create explicit height and width (it is better to avoid it or set min-height, max-width depends on a project);
- don't style on tags (that's a reason why you have got classes);
- classes are not very readable => I'd recommend learning BEM naming convention;
- check your project by the inspector in your browser on different devices (especially margin, footer, padding).
Greetings :D
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