four-card-feature-section-master

Community feedback

• 

  P

  Valeria Montoya•400

  @ValeriaMontoyaPosted 3 months ago

  Hi @Elofatish 👋🏼

  There are some positive aspects about your solution, but also areas for improvement:

  • You correctly use alt attributes for your images, which is great for accessibility! However, for images like "Supervisor", "team-builder", etc., you might want to make the alt text more descriptive. For example, instead of alt="Supervisor", use something like alt="Icon representing Supervisor tool for monitoring project roadblocks". This helps users with screen readers understand the context.
  • Avoid unnecessary comments like "file css" as it's self-explanatory from the link tag (line 10 of your index.html file).
  • Finally, your implementation looks close to the design but some colors and font sizes can be changed to match the design. For example, you can replace color black for --Very-Dark-Blue: hsl(234, 12%, 34%);

  I hope you find my feedback useful! 🤓

  0