Responsive Order summary component

Community feedback

• 

  teamie•470

  @tea-scriptsPosted over 2 years ago

  Great job completing the challenge. Some points

  • You might want to consider using an article instead of a section. This is somewhat a card and not an entire section. an article can be placed inside of a section and also vice-versa.
  • Another thing missing is a transition on your buttons. You want the :hover to take effect over time and not instantly. This makes your design more pleasing to the user and gives better experience.

  You did an amazing job. Happy coding 👍!

  Marked as helpful

  1

  ---

  

  Joshua•130

  @dyntbnPosted over 2 years ago

  @coder-teamie

  So glad you told me that, because I've been doing that for a while. 🙃

  By the way, is there a particular resource that helped you understand HTML element placement better? Right now, I'm using MDN Docs and sometimes YouTube videos for help.

  Also you're right about the transitions, going back through it right now, it does look plain.

  I'll make both of those changes in the Github repo, and thank you so much for the pointers Teamie. 😄

  1

  

  teamie•470

  @tea-scriptsPosted over 2 years ago

  @dyntbn That’s great. The resource I use to understand the HTML semantics is W3Schools HTML Semantic Tags. This resource is self explanatory with examples and yes the MDN is also another wonderful resource to learn better about HTML elements.

  Marked as helpful

  1