Results-summary-component

Community feedback

• 

  Mike Stop Continues•100

  @mikestopcontinuesPosted 11 months ago

  Hey, this is very nice work.

  I like that you're getting some experience with tailwind. It's very popular these days. That said—consider it a stepping stone to learning to write good pure CSS. CSS libraries only last a few years each, and your career will be much longer than that.

  Also, designers can be very picky about their work. In the workplace, I would give this one more pass to make sure the width and the purple-blue gradient was the same as they asked for. (I'm sure you know this, but you can use a color picker to grab the top and the bottom gradient colors to create your linear-gradient() css style.)

  Lastly, I would think about making your HTML more semantic:

  • This widget would likely be part of a larger page. That makes it a section.
  • Similarly, we generally only see one h1 per page, so it would be best if this widget started with an h2. You also want to make sure that all other headings are subservient to the main heading of your widget. So all of the others should be h3 or below.
  • Third, your results are really more of a list. Rather than use headings, consider using ul and li to indicate they are a group, and that the elements are "parallel" to one another.

  You're destine for great things.

  Marked as helpful

  0

  ---

  

  Imran•150

  @xdevimranPosted 11 months ago

  @mikestopcontinues Thank you so much for your kind words. I'm also planning to learn more deep CSS and the next time, I'll focus on the semantic HTML. Thanks again, have a good day @Mike

  1