Heli0s• 670
@zeerobit
Posted
- you only need grid-template-areas on the main container and 4 columns for your grid-template-columns to achieve this layout
- the position property is not needed in this challenge, i noticed you used it for the photos and the quote, remove the quote img from the html and add it in css as background then use background-position to move it
- you use % unit for padding i would not advise it, use rem instead
- px is not scalable , replace it with rem
- too many divs, you should use proper html semantics such as figure, figcaption, blockquote to wrap your content
- why using % for width, height and padding i think all of this percentage is causing issues, you'll notice your layout breaks once it hits about 772px
hope this help... Happy coding
2