Design comparison
Community feedback
- @NehalSahu8055Posted over 1 year ago
Hello Coder 👋.
Congratulations on successfully completing the challenge! 🎉
Few suggestions regarding design.
➨ Remove
commented lines
from the below code of your design as this is not good way to center the card.card { /* position: relative; */ /* top: 200px; */ /* margin: 0 auto; */ }
➨ To properly center the container.
- USING FLEXBOX
body{ min-height: 100vh; display: flex; align-items: center; justify-content: center; }
- USING GRID
body{ min-height: 100vh; display: grid; place-items: center; }
- Use
cursor:pointer
for buttons for more user friendly.
I hope you find this helpful.
Happy coding😄
Marked as helpful0 - @BKvipPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉 I have other recommendations regarding your code that I believe will be of great interest to you.
- responsive moniter by code
@media only screen and (max-width: 600px) { #card{ flex-direction: column; } }
link: https://www.w3schools.com/css/css_rwd_mediaqueries.asp
I hope you find this helpful
Happy coding!0@OversrPosted over 1 year agoThank you for the feedback, it’s greatly appreciated. @BKvip
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