@PJIceskull
Posted
Hey @Joolaoluwa, one thing I noticed about your solution is missing the color
, background-color
, and font-style
used in the design. In the starter files provided by Frontend Mentor there should be a style guide that lists the color values and fonts used in the design. This information is in the Colors
and Font
sections.
Another thing I wanted to point out is how your project's set up. In your repository, you have three CSS files, usually, it's a better idea to have your CSS code in one folder for file organization. I also don't think you need to create additional CSS files and link them using HTML to do Media Queries. In your own CSS file, you can use the CSS media
rule when working with media queries. Code Example:
@media screen and (min-width: 400px) {
// Insert Code here.
}
Other than that you did a good making the website more responsive.
@Joolaoluwa
Posted
Thank you for your feedback, I would make the changes to the styling and the css files too @PJIceskull