3-column-preview-card-component

Community feedback

• 

  Mayank Arora•430

  @mayankdrvrPosted about 1 year ago

  Congratulations @corolaweb023 for completing this challenge. Your design matches the solution very well and your code is following good practices.

  Here are a few observations-

  1. The <h1> element should be used for SEO atleast once, but it should be used only once per page.
  2. Try to use the Block Element Modifier(BEM) naming method as a good practice of naming classes for referencing html elements in the css file.
  3. Avoid using <div> element in html file and use semantic html elements throughout the code for better web accessibility.
  4. Using noopener and noreferer in <a> elements is a good security practice with cross browser compatibility.
  5. Use the page background color given in the starter files.

  Awesome solution and keep it up.

  Marked as helpful

  1

  ---

  

  corolaweb023•20

  @corolaweb023Posted about 1 year ago

  Thank you for the feedback @mayankdrvr

  1
• 

  Lukas Rucky•160

  @malyrudyPosted about 1 year ago

  good job 👏 and I have checked also the code css and its awesome DRY semantic code

  1

  ---

  

  corolaweb023•20

  @corolaweb023Posted about 1 year ago

  Thank you for appreciating my work @malyrudy

  1