revanthv999• 110
@revanthv999
Submitted
I hope it is close....
RNGideon
@RNGideon
All comments
- RNGideon• 20
@RNGideon
Posted
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
@Lb2468
Submitted
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
RNGideon• 20@RNGideon
Posted
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 - Ppriyankac79• 70
@priyankac79
Submitted
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!
RNGideon• 20@RNGideon
Posted
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
