@VCarames
Posted
Hey @jonexist, some suggestions to improve you code:
- You do not want to have or use empty divs in you HTML code, since this is considered to be bad practice. Instead use the Picture Element
Picture Element will allow your to switch between images in different breakpoints
Syntax:
<picture>
<source media="(min-width: )" srcset="">
<img src="" alt="">
</picture>
Source:
https://www.w3schools.com/html/html_images_picture.asp
https://web.dev/learn/design/picture-element/
-
The stat numbers are not headings. They should be wrapped in a Paragraph Element.
-
The Section Element is being used incorrectly. Instead and Article Element will serve as the component's container and each side should be a simple Div with a class.
-
Your content is not fully responsive. Here is a link to Google Developer’s site that will teach you how make it 100% responsive:
Happy Coding! 👻🎃
Marked as helpful
@jonexist
Posted
@vcarames thank youuuu for correcting some of my error! And thank you for the useful information that you gave!