Design comparison
Solution retrospective
Any feedback would be really great! Thanks
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi fsuropaty, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
I noticed that in higher resolutions the content is stretching a lot, and this is affecting the design of the page, to fix this we can define a
max-width
in the content that we want to have a maximum width! thus:@media only screen and (min-width: 768px) html, body { max-width: 1440px; margin: 0 auto; }
To improve the code structure wrap this div:
<div class="attribution">
with the semantic tag
footer
The rest is great!
I hope it helps...
1 - @Mohammedsalih1Posted about 2 years ago
awesome design dude, i took a look at it, it is extremely perfect
Heads up1@fsuropatyPosted about 2 years ago@Mohammedsalih1 Thank you so much, really appreciate it
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