Design comparison
SolutionDesign
Solution retrospective
it looks quite large ,,,how to fit it on whole screen?
Community feedback
- @SzymonRojekPosted almost 4 years ago
Hi Mansi,
Well done! :D
I've just checked your HTML structure and preview site by the inspector. A few tips from me:
- I think, the article tag can not be used here as a general wrapper for your component. I'd recommend getting more information about semantic tags here;
- "Verified Graduate" doesn't have to be a heading, what do you think about a paragraph?
- maybe you can see each box as an article with the header, also you can check the blockquote tag;
- your HTML is not very readable, it will be better if you write it more structurally with spacing.
If you ready to learn Responsive Web Design, I can recommend Flexbox & Grid courses made by Wes Bos, they are for free:
Finally, congrats! ** Please, don't forget to upvote any comments on here that you find helpful. That's it from me. Hopefully, it will help you.
Greetings :D
3@mansi05041Posted almost 4 years ago@SzymonRojek thanks a lot . i will make changes sooner
0 - @AryaAnish121Posted almost 4 years ago
Hi Mansi,
Nice two suggestion:-
- You could give
.container {transform: scale(.75)}
- You could make it responsive
Happy Coding!
2 - You could give
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