Peter Jacobs• 190
@thisispeterj
Posted
Your solution to the challenge looks really good.
- The HTML is semantic, I don't see anything that I would update.
- I like your use of bem for the css to keep it organized.
- The setup of the root custom css properties is great if this project were to ever expand in size.
- The use of the grid template areas is great, and as someone peer reviewing the code, makes it very easy to understand and follow.
Overall very nice work!
0