FM Order Summary Component

Community feedback

• 

  Saad Hisham•1,770

  @Saad-Hisham

  Posted about 2 years ago

  Great job❤ I have some tips that could help make your work even better. Firstly, for better accessibility, consider wrapping the card in a <main> tag instead of a <div>. This will make it easier for all users to navigate and understand your content.

  Secondly, the alt attribute of your image should be more descriptive. For example, you could describe the image as "a young girl wearing headphones, dancing and enjoying the music". This is important because some users may rely on screen readers to access your content, and providing a clear description will help them understand the image.

  Lastly, for the design, consider adding the following code to make the background match the design:

  body {
  background-repeat: no-repeat;
  background-size: contain;
  }
  

  By replacing the cover value with contain, the background will better fit the design of your work.

  Overall, your solution is great Keep up the good work🔥

  10