@aguscorvo
Posted
Good job on solving the challenge! I saw that you already fixed the rounded corners on the illustration image—well done!
Here are some suggestions to improve the solution or tips moving forward:
- The title is missing a hover state.
- How about changing the p tag with the class title to an h1 tag?
- Instead of adjusting margins or using unnecessary padding on each element, you could use display: flex and gap. This approach also makes it easier to match the designs. When you start styling components, you’ll notice the benefits of this.
- For future projects, if you have the chance to check the Figma designs, take advantage of it to ensure the sizes and other properties match.
- I’m following the “Getting started on Frontend Mentor” path, and there’s an interesting challenge: “The font sizes in this project are slightly smaller in the mobile layout. Find a way to reduce the font size for smaller screens without using media queries.” It’s fun to solve!
- One more tip for future projects: folders that start with a dot, like .vscode, are generally not meant to be included in the repo. I suggest adding them to the .gitignore file to avoid issues when working with other developers.
Marked as helpful
@franklobsty25
Posted
@aguscorvo Thank you for the feedback. I am grateful and will do what was suggested.