Moozika Order Component Summary

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted about 3 years ago

  Hey, its a good looking solution! Font-size could be a bit bigger to make it easier to read. The container div could be swapped for main and the Order Summary title for h1 to make it more semantic. ^^

  Marked as helpful

  0

  ---

  

  Alexander Craggs•35

  @EyreCPosted about 3 years ago

  @FluffyKas done! I learned from this that the decision to use h1, h2 or otherwise is largely semantic as opposed to size-specific. I also learned about the <main> tag, which I understand to be a focal point of interest in the html. Thank youf ro the feedback!

  1
• 

  Chiaza Okoli•70

  @iKINGDAVIDPosted about 3 years ago

  Not pixel-perfect, but accurate enough. Why use tailwind tho? This could too easily be done with pure CSS. I saw what looked like a misplaced div closing tag (without an opening pair) on line 44. I don't really see any mistakes. Looks good, well done.

  0

  ---

  

  Alexander Craggs•35

  @EyreCPosted about 3 years ago

  @iKINGDAVID Thank you for the feedback! I was looking to gain some experience using tailwind as a framwork. No specific reason to use it for this project from a technical perspective, and pure CSS would probably have been better for a project of this scale.

  0