I use the html and css for this website.This is fully responsive .
Design comparison
Community feedback
- @khushi0909Posted over 1 year ago
1)make width of container card bigger and add padding ,maybe you have done to make it mobile first or look even at every screen ,but little big with padding in all texts will look better and may be then you have use little media queries for responsiveness to fit it in all screens
2)Read about h1 tag its the most important and every website should have at least one https://www.semrush.com/blog/h1-tag/
3)you should learn about semantics HTML and using it for ex footer ,main etc
All the best
1@TejasNasrePosted over 1 year ago@khushi0909 my design is perfect only size is small that's all
0 - @MohamedatePosted over 1 year ago
if you include font family and padding for all text from right and left and make width of card more bigger than this it will become like the original
1@TejasNasrePosted over 1 year ago@Mohamedate i don't waste time on the font and all that yes the size is small but doesn't matter and thanks for feedback 🙂
0@WeberowskyPosted over 1 year ago@TejasNasre it's not a waste of time. You could've just used different font. Also you didn't add the shadow. The fonts and colors are in the
style-guide.md
file and using them in your projects doesn't take too much effort.1@0xabdulkhaliqPosted over 1 year agoBrother @TejasNasre, I'm gladly welcome you to the frontendmentor community
People are here to do constructive criticism, they are not making fun of your skills.
We want to hear fellow developers suggestions to improve our skills, being rugged against feedback providers is not a good practice and it's need to be strictly avoided inside the community
Thanks
Marked as helpful1
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