nft-preview-card-component-main

Community feedback

• 

  Algka•200

  @Algka

  Posted over 1 year ago

  Thank you for our suggestions. I like more that

  00
• 

  Abdul Khaliq 🚀•72,360

  @0xabdulkhaliq

  Posted almost 2 years 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.

  ---

  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

  ---

  • To properly center the component in the page, you should use Flexbox or Grid layout. You can read more about centering in CSS here 📚.

  ---

  • For this demonstration we use css Grid to center the component.

  body {
      min-height: 100vh;
      display: grid;
      place-items: center;
  }
  

  • Now remove these styles, after removing you can able to see the changes

  .rectangle {
    margin-left: 550px;
    margin-top: 100px;
  }
  

  ---

  ---

  • Now your component has been properly centered

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  00
• 

  Jonah Ssegawa•240

  @devjhex

  Posted almost 2 years ago

  @Algka You have done a very great job on this component, It shows that you put in a lot of effort completing it, Well done. I hope this suggestion would help in the positioning of the component in the center of the page.

  body{
       background-color: var(--Very-dark-blue1);
       display: flex;
       justify-content: center;
       align-items: center;
       flex-direction: column;
       height: 100vh;
  }
  

  Otherwise, you have done an excellent job. Keep up with the code!!

  00