All comments

• 

  Cj Rivas•50

  @CjlikescodeSubmitted almost 2 years ago

  Really liked the small component! Fun exercise to practice on.

  

  Responsive QR Component

  1

  20
  

  edbizichjr•100

  @edbizichjrPosted almost 2 years ago

  Great job!

  Only comment I have is that for your first section that houses everything you use the <Main> tag since it is what is housing the main content.

  Happy Coding!

  Marked as helpful

  0
• 

  Ralph•190

  @delacruzralphSubmitted almost 2 years ago

  What are some best practices when it comes to centering elements?

  

  QR code component

  1

  10
  

  edbizichjr•100

  @edbizichjrPosted almost 2 years ago

  Hello!

  Card looks good!

  Tips for centering elements I personally have found flex box to be helpful. On these projects I often use it to center it since they are just small elements. More can be found out here. https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Flexbox

  Another note is to try to always link out to your stylesheets to get into best practices and keeping them out of your html.

  Happy Coding! -Ed

  1
• 

  Makia Henderson•40

  @Makia98Submitted almost 2 years ago

  Please feel to leave suggestions, especially pertaining to HTML Structure and CSS styling.

  

  BootStrap Order Summary Card

  1

  20
  

  edbizichjr•100

  @edbizichjrPosted almost 2 years ago

  Good job on completing this challenge Maria! The only things I noticed where the hover states were not there on the live site for the active items in their documents provided.

  Great job on utilizing bootstrap and mdn as resources and keep up your journey.

  0