NFT preview card component using CSS custom properties and CSS flexbox

Community feedback

• 

  Yash•130

  @yashviradiaPosted over 2 years ago

  Hi!

  Great work on this project! :)

  Here might be some points to consider:

  1. better to write CSS and HTML codes in separate files, HTML generally in index.html and CSS in styles.css
  2. <p> instead of <P> for paragraph tag
  3. for CSS selectors use new names to avoid confusion with HTML tags
  4. cursor: pointer; can be used for focusing image, heading, and creator's name

  Hope that helps and happy coding!

  Marked as helpful

  0

  ---

  

  Enoch•20

  @Sh0lexPosted over 2 years ago

  @yashviradia Thanks for the feedback it is appreciated

  1
• 

  Hamza-Noah•300

  @Hamza-NoahPosted over 2 years ago

  Hello

  here is my feedback on your solution first, increase the padding around the content of the card. second, try to increase the border radius of the image and the card itself. and for the last thing to edit in the design the challenge by front end mentor should be at the bottom of the page.

  after that check the report and follow the steps one by one until you have 0 issues. hope that helps

  Marked as helpful

  0

  ---

  

  Enoch•20

  @Sh0lexPosted over 2 years ago

  @Hamza-Noah Thanks for the feedback.

  0