Martin• 1,150
@ringm
Posted
Hey Amokomowo, great job. It looks fantastic, and it completely responsive, congratulations on finishing this one!
The only thing I can think to polish it would be to remove the fixed height on the main element of the html. When the device screen starts to get narrow, and before reaching the mobile layout, you get a lot of unnecesary white space on the first box. I think the design would improve if you leave the height be determined by the content. If your aim is to center the component on the middle of the screen, you can achieve it with display: flex; on the parent and margin: auto; on the element you want to center.
Great work! Keep it up! ;)
1
Amokomowo Bashir• 260
@Barsheyr
Posted
@ringm Thank you so much This means a lot
1
