DmitryIsTrying
@DmitryIsTryingAll comments
- @Rajeev0422Submitted 28 days ago@DmitryIsTryingPosted 28 days ago
It would be better to wrap the following in a separate container:
<h2>Improve your front-end skills by building projects</h2> <p>Scan the QR code to visit FrontEnd Mentor and take your coding skills to the next level</p>
Using
padding-left: 0px; padding-top: 20px; padding-bottom: 20px; padding-right: 0px;
is generally not a good practice because it’s inconvenient. Read about how to set padding for all sides using justpadding: 1px 1px 1px 1px;
(for example).Don't write margin: 0; and padding: 0; for every block; instead, use:
* { padding: 0; margin: 0; }
This way, you’ll set zero margins for all elements, and then override them where necessary.
The image sizes are incorrect. The height should specify px.
The design uses the Outfit font, and it needs to be downloaded from Google Fonts. Read about how to do this!
Overall, try not to hardcode the width and height, as this is generally a bad practice for the responsiveness of the website.
0