NFTPreviewPardComponentChallengeHub

Community feedback

• 

  iago•400

  @iagohenrique2009Posted over 2 years ago

  Hi!

  There are a few changes that you can do to make your work even better!

  1- Your css code is really really long , you can do it become more readable and easy to see just by using a 5 divs{1 to container, 1 to mid text ,1 to the icons, 1 to bottom}. give a read to this site if you get stuck (https://css-tricks.com/snippets/css/a-guide-to-flexbox/);

  2- You has given a unnecessary margin to body;

  3 - remove your div called linea and place an <hr> with the css properties: hr{ border:0; border-bottom:1 px solid hsl(215, 51%, 70%); width:80%}

  4- replace the ending span for a div and remove the spans of image and text and for the text create a <p> tag and only 1 span for " Jules Wyvern"

  Marked as helpful

  0

  ---

  

  Carmen•490

  @CarmenyoPosted over 2 years ago

  @iagohenrique2009 thank you very much for your help. I will do it as you recommend.

  0