All comments

• 

  KEERTHI B•20

  @keerthi797Submitted about 1 year ago

  All feedback is welcome

  

  Product preview card component

  2

  40
  

  Khalid Khabouz•190

  @KhalidKhabouzPosted about 1 year ago

  Nice job, just try to use semantic elements as a habit for the long term, to help the search engines and other user devices to determine the importance and context of web pages.

  Generally you did a good job keep up.

  Marked as helpful

  0
• 

  Oussama Taali•250

  @oussamataali97Submitted almost 3 years ago

  

  Responsive profile card for desktop and mobile devices

  1

  30
  

  Khalid Khabouz•190

  @KhalidKhabouzPosted almost 3 years ago

  The layout looks nice, good job for this one. But I just have some remarks to improve the accessibility issues.

  1. Try to wrap your section element inside the main tag.
  2. Try to wrap the attribution div inside a footer tag.

  Also to improve the typography of your design, use the font provided in the style-guide.md file.

  Good job and happy coding

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Khalid Khabouz•190

  @KhalidKhabouzSubmitted almost 3 years ago

  I've just completed this challenge, it was helpful. Thanks for stopping by.

  

  nft preview card component HTML & CSS

  2

  20
  

  Khalid Khabouz•190

  @KhalidKhabouzPosted almost 3 years ago

  That was so informative and helpful, you are right, it is better to build good practices as a foundation. Thank you for your time.

  0
• 

  Khalid Khabouz•190

  @KhalidKhabouzSubmitted about 3 years ago

  This is my solution for this challenge. It was a good thing to work with multiple background images using CSS. Thank you for stopping by.

  

  Profile Card Component HTML, CSS, SCSS

  2

  20
  

  Khalid Khabouz•190

  @KhalidKhabouzPosted about 3 years ago

  Thank you so much for all these remarks, it was very helpful for me to build and develop a good practice. Thanks again for your time for stopping by.

  1