Design comparison
SolutionDesign
Solution retrospective
All feedback are welcome
Community feedback
- @kimodev1990Posted 11 months ago
Great work in completing the challenge, Just a few feedback :
- To center your main-container in the middle of your website, you could add in body :
display: flex ; justify-content: center ; align-items: center ; min-height: 100vh ;
then the container will be centered, no need to use
margin-top: 100px ;
- Wrap your design in main tag.
- Instead of assigning definite values to
width: 250px ;
&height: 250px ;
for div class .article & div image, It's better in future designs to assign percentage values for flexible & responsive design. For example: you could assignwidth: 50% ;
for each one them , So they would in relative to the div class .main-container size. - You could use clamp ( ) method in your coding for font-size, width, margins, padding, etc., So the designed sizes will change according to the viewport dimensions having a responsive design and will be suitable for any device layout.
Hope you find this Helpful.
Other than that, Really Nice work & keep Going on
Marked as helpful0 - @exist08Posted 11 months ago
good work !
keep doing the good.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord