Design comparison
Solution retrospective
Responsive side was hard, but anyway I am proud of it. It looks like a magic one.
What challenges did you encounter, and how did you overcome them?Adding cross-line on the element was hard, but I've found it using google ;)
What specific areas of your project would you like help with?Any feedback will be helpful. Thank you in advance.
Community feedback
- @01057057kimPosted 8 months ago
I noticed that your card is still not in the middle of the page. you can apply this to the body:
body { min-height: 100vh; display: flex; justify-content: center; align-items: center; }
I hope this helps. 😁
Marked as helpful1@ursgolPosted 8 months ago@01057057kim Thank you so much for your feedback :) I will try to improve it in my free time :)
0 - @Harsh-Kumar-DwivediPosted 8 months ago
Hi @ursgol ,
I think it would be better to use
<div>
instead of<h6>
in the following:-<h6>Perfume</h6>
Headings are meant to be used in hierarchical order not for the purpose of text size or weight or both, is what I know.
Hope it is helpful !
Happy Coding !
Marked as helpful1@ursgolPosted 8 months ago@Harsh-Kumar-Dwivedi thank you for your feedback. You are right. I will try to improve it :)
1@Harsh-Kumar-DwivediPosted 8 months ago@ursgol
Welcome !
Keep up the good work !
Happy Coding !
1@ursgolPosted 8 months ago@Harsh-Kumar-Dwivedi Ok, I will. Thank you so much. Happy Coding for you too :)
1
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