Skip to content
OverviewSolutions
0
Comments
3
RNGideon
@RNGideon

All comments

  • RNGideon•20
    @RNGideonPosted 10 months ago

    Try to do the mobile view as well.

    I don't think you need set a height on the container -- the content will determine that. Try switching from px to rem/%. Just use px when absolutely have to and there's a good reason for it -- otherwise use rems/%/etc.

    Marked as helpful
    0
  • LB•60
    @Lb2468Submitted 10 months ago

    Hi I tried to solve the problem but I know my code is not perfect, please if you can help me to make it very professional, I will be happy

    recipe page

    2
    RNGideon•20
    @RNGideonPosted 10 months ago

    Try to fix the inconsistencies with your padding -- horizontal rhythm.

    And change the way you handle the responsiveness -- don't do it like that. Just set a max-width to the wrapper, margin auto the left and right (to center) -- and then padding left right so the content is not touching the sides on narrow screen sizes.

    0
  • priyankac79•70
    @priyankac79Submitted 10 months ago

    Hey Guys, I hope you all doing great, I completed the challenge, and I hope it's good enough to like you!

    Any suggestions are welcome!

    Recipe Page Using CSS Grid

    1
    RNGideon•20
    @RNGideonPosted 10 months ago

    The body background is supposed to be the eggshell color, right? and then the content area is on top of that with the white background.

    0