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

Responsive page using media query and layouting using css flexbox

@gilangr1440

Desktop design screenshot for the Recipe page coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I can copy the design almost the same

What challenges did you encounter, and how did you overcome them?

set class naming and determine css specificity

Community feedback

P

@ePaulo

Posted

Class naming is excellent. Responsiveness is very good. A couple of suggestions:

  1. Consider using a CSS universal selector (*) to match elements of any type when resetting margin, padding, border, and so at the top of your CSS code. See the MDN Web Doc... https://developer.mozilla.org/en-US/docs/Web/CSS/Universal_selectors
  2. Consider adding line-height settings to content (i.e. the space between lines in a paragraph) to match the design a bit closer. See the MDN Web Doc... https://developer.mozilla.org/en-US/docs/Web/CSS/line-height

Marked as helpful

1

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