HTML 5 & CSS

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted about 4 years ago

  Hello, Harold! 👋

  Good job on this challenge! Your background solution seems okay. It only needs small changes.

  • Try work with background-size and background-position of your background to fit design. You set size to 100% and in design they're much smaller, then position them once again.
  • <hr> element represents a thematic break between paragraph-level elements. Maybe add border-bottom to .profile div element instead. That way your solution will be closer to design.

  Good luck with that, have fun coding! 💪

  1
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Harold! 👋

  Good job on another challenge! 👍

  I'd like to suggest using flexbox to center the card component in the middle of the viewport (rather than using a bunch of margin/padding-top) so that it remains in the center even when the screen is resized. Or you could use margin: auto to do the same thing, if you'd like!

  Keep coding (and happy coding, too)! 😁

  0
• 

  Harold•628

  @harika09Posted about 4 years ago

  This is noted. Thanks!

  0