Design comparison
Solution retrospective
Does the responsive mode incorporate the design? What aspects of the design lack what the actual design has?
Community feedback
- @PJIceskullPosted over 1 year ago
Hey @Joolaoluwa, one thing I noticed about your solution is missing the
color
,background-color
, andfont-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 theColors
andFont
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.
1@JoolaoluwaPosted over 1 year agoThank you for your feedback, I would make the changes to the styling and the css files too @PJIceskull
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord