Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive nft preview card component using HTML and CSS

Franceβ€’ 100

@LanXhan

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello! Frontend Mentor community! Day 6 and this is my solution for nft preview challenge for my daily challenge to code and learn more about front-end development.

Happy to hear feedback and advice!

Community feedback

Abdul Khaliq πŸš€β€’ 72,660

@0xabdulkhaliq

Posted

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.

CSS 🎨:

  • Looks like the component has not been centered properly. So let me explain, How you can easily center the component without using margin or padding.
  • We don't need to use margin and padding to center the component both horizontally & vertically. Because using margin or padding will not dynamical centers our component at all states
  • The another reason why your component is not being placed on center is the incorrect usage of width and height, You want to use min-height: 100vh. Since you already using Flexbox layout in css, So take a look the example code
body {
min-height: 100vh;
}
  • Now remove these styles, after removing you can able to see the changes
body {
margin: 0 auto;
margin-top: 20vh;
}
  • Now your component has been properly centered

.

I hope you find this helpful πŸ˜„ Above all, the solution you submitted is great !

Happy coding!

0
_nehalπŸ’Žβ€’ 6,710

@NehalSahu8055

Posted

Hello Coder πŸ‘‹.

Congratulations on successfully completing the challenge! πŸŽ‰

Few suggestions regarding design.

  • For proper positioning of nft eye(view-icon) delete the commented line and replace transform with the below code
.img-overlay img {
position: absolute;
/* left: 65px; */
/* top: 50px; */
transform: translate(7.5rem,7.5rem);
}

I hope you find this helpful.

Happy codingπŸ˜„

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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