Design comparison
Solution retrospective
Summary In this "nft-card" project, a whole "card" class is introduced, where inside there is an image, a title, a paragraph, as well as a sub-class, the location of which is aligned with the help of "flexbox". And finally, I used the "hover" effect.
Community feedback
- @Tux3er-IsmaPosted almost 2 years ago
Hi Kapandaze!!!
I am Tux3er 🐧, a junior frontend dev and I have reviewed your code
Your code looks nice, congrats 👍🏻🎉
I will give you some tips for your next project 😉💡
-
Try to use BEM css methodology. There is some documentation if you want to read or if you want to watch a video I also will put you a video. Documentation 📖 Video 📹
-
If you want to use css reset use normalize.css 🎨 file.
-
In your readme.md file 📖 in your Github Repo 😺 put more information about your project like the tecnologyes or something like this.
-
In the img hover states put some
cursor: pointer;
to get a better user experience. -
Don't forget to put who did the project at the end of the page.
-
In the responsive mobile put some
margin-top
at the card. You can do this by using media-queries.
I hope this comment will help you a lot in the future 🖐🏻🌟⌨️
Marked as helpful0@Kapanadze04Posted almost 2 years ago@Tux3er-Isma Thank you very much, I will definitely take your advice into consideration.
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