Szymon Rojek• 4,540
@SzymonRojek
Posted
Hi Danny,
Well done! :D
I have checked your HTML structure, a few tips for you:
- instead of div container maybe you can use the main tag here, inside can be one section with 3 divs => in this example divs are perfect for generic groupings of content;
- the header tag is not needed here IMO => it is very easy to overuse tags;
- I don't recommend to use the h1 for price. Just to let you know, you should only use one h1 per page. Using more than one will not result in an error, but using only one is seen as a best practice. => h1 is the most important heading, and tells you what the purpose of the overall page is (generally please read about headings h1-h6). I know this is a small project but consider it as one unique single page example (h1 and h2 in inside first box and h2 another boxes);
- button => please check this article from the blog CSS tricks: A Complete Guide to Links and Buttons;
- check your project on small devices by inspector in your browser (give a margin);
- check your HTML issues report above and try to fix it.
Greetings :D
1
