NFT preview card component

Community feedback

• 

  Jo•110

  @joana-trotsPosted about 2 years ago

  Good work! I see you’re having trouble centering <main> element, try to add Flex Box properties for body { height: 100vh; display: flex; justify-content: center; margin: 0; padding: 0; }, so for <main> block you don’t have to use margin-top property, and it’s beautifully centered!👌 You can also use Flex Box properties to place elements inside 'container' block (without using position: absolute). You can learn about flexboxes here: https://css-tricks.com/snippets/css/a-guide-to-flexbox/ or anywhere else 😊 Hope it was helpful! 🤘

  Marked as helpful

  0

  ---

  

  ilyasazer•100

  @ilyasazerPosted about 2 years ago

  @joana-trots thanks for your help, and i will work on it 😊

  0
• 

  Azeem Saifi•370

  @saifion33Posted about 2 years ago

  Hi ilyasazer.

  You overall design is great .There is only one accessibility issue you should use atleast a level one heading

  so use <h1> tag instead of <p> tag.

  <h1 class="p1">Equilibrium #3429</h1>

  Happy coding.

  Marked as helpful

  0

  ---

  

  ilyasazer•100

  @ilyasazerPosted about 2 years ago

  @saifion33 Thanks for your advice, I will work on it 😊

  1