CSS Flex-box, Responsive web design

Community feedback

• 

  Pierce Issah•120

  @PJIceskullPosted over 1 year ago

  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.

  1

  ---

  

  Joolaoluwa•10

  @JoolaoluwaPosted over 1 year ago

  Thank you for your feedback, I would make the changes to the styling and the css files too @PJIceskull

  0