Responsive log in page using HTML5, CSS Media query and little flexbox

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hey Henry, congratulations on completing the challenge! You did a great job 😉

  Let me give you some little tips for optimizing your code:

  • to fix the background image size remove from body the background-size: 1440px 700px property
  • add main tag and wrap the card for improve the Accessibility
  • add descriptive text in the alt attribute of the images
  • not use <br> to separate elements because it's for break the line of test, use margin property
  • remove all margin from .content class
  • use flexbox to the body to center the card. Read here -> best flex guide
  • after, add min-height: 100vh to body because Flexbox aligns child items to the size of the parent container
  • instead of using px use relative units of measurement like rem -> read here

  Hope this help! Happy coding 😁

  Marked as helpful

  0
• 

  idukpaye alex•725

  @Esesosa-maxPosted over 2 years ago

  1. ✅ Good Looking Design 3.94/5
  2. ✅ Responsive 3.9/5
  3. ✅ Good Attempt (See Way worse) 3/3
  4. ❌ HTML Not Properly structured 2.5/5
  5. ❌ Card is not properly Centerd
  6. ❌ unsuitable Font

  Marked as helpful

  0

  ---

  

  Henry Agu•240

  @HenryAguPosted over 2 years ago

  @Esesosa-max thank you

  0

  

  idukpaye alex•725

  @Esesosa-maxPosted over 2 years ago

  @HenryAgu , please you can mark it as helpful

  Marked as helpful

  0