Design comparison
SolutionDesign
Solution retrospective
Hi everyone. I am new to this, please feel free to check my codes and comment suggestion to make my codes neat, correct, and short. Thank you.
Community feedback
- @SzymonRojekPosted almost 4 years ago
Hi Georgee,
Well done :D
I have mainly checked your HTML structure, a few tips from me:
- we can not skip the headings so "30-day, hassle-free money back guarantee" can become h2;
- I'd recommend reading more about semantic tags (which can use here), headings (why they matter);
- instead of a button I suggest to use a link;
- in the div class="rightbox" instead of it use ul > li tags (using p and br like that it is not a good practice);
- names of the classes are not very readable, it will be difficult to navigate in CSS file;
- project is not responsive on different devices => did you start to learn flexbox and grid?
Greetings :D
1@iamgorgeePosted almost 4 years ago@SzymonRojek Hi. Thank you for the tips. I will try your suggestions.
0
Please log in to post a comment
Log in with GitHubJoin 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