@Shahbaaz92
Posted
Hello, The site was good but do not use % or rem for paddings and margins instead use ems or px for paddings/margins and reduce the vertical gap between the body and the container. Let the width of the first article(first child of the main container) and section (second child of the main container) be 100% of the main container, that way it will be responsive. In the section element after the display: flex use justify-content: space- evenly, so that they are spaced evenly.
To use grid .flex-container{ display:grid; grid-template-rows: 1fr; grid-template-columns: repeat(2, auto) } .flex-container article{ grid-row: 1/-1; } .flex-container section{ grid-row: 1/-1; #here you can use flexbox instead of grid for the children }
Marked as helpful
@Omowunmikamil
Posted
@Shahbaaz92 This was helpful, I will apply the corrections. Thank you!