@Chinex4

Jech
@j3rech0All comments
- @j3rech0
Is this the final output? It looks to me that you didn't implemented responsive layout on this challenge. I don't see any
@media
query in the source code. - @SayHelloToKam@j3rech0
Here's a few noticeable:
- Avoid using fixed value in a container especially setting the
height
for the class name.outside-right-container
, that section will have a dynamic content I assume,height: auto
I guess is okay. - Also dealing with
width
for example in themain
you could havewidth: 100%;max-width: 400px;
for better scalability. - Also I think mobile-first approach is still relevant. Overall good job!
Marked as helpful - Avoid using fixed value in a container especially setting the