Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

CSS Flex-box, Responsive web design

@Joolaoluwa

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Does the responsive mode incorporate the design? What aspects of the design lack what the actual design has?

Community feedback

@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.

1

@Joolaoluwa

Posted

Thank 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 GitHub
Discord logo

Join 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