@MohamedHussein05
Posted
Great work on this one, I'll just Suggest few things to improve your work:
-
The Card doesn't appear properly in the Landscape mode using
body { (min-height: 100vh;) }
instead of the current rule and removing#bottom-section { (height: 70%;) }
will fix it. -
It's not recommended to set height on the card Use paddings & margins instead.
-
The scrollbar isn't very nice, disabling it will give your page a better look.
-
Try as hard as you can not to use height .. but If you must use min-height.
Marked as helpful
@jimavictor
Posted
@MohamedHussein05 Thanks a lot for the feedback. i will make these changes as soon as i can..