html and css

Community feedback

• 

  Grego•1,310

  @Grego14Posted 6 months ago

  👋 Hello! 🎉 Congratulations on completing the challenge! 🎉

  Remember to specify a more descriptive alt attribute, as it is commonly used by screen readers, no one wants to listen to 'photo profile'. Something like this 'Jessica Randall photo profile' would look better.

  You're using two media queries unnecessarily. You have two for 600px and two for 750px.

  Move the media querys to the end of the file and add the styles from the a element to the media query where you add the styles to the .main-container.

  @media screen and (min-width: 750px) {
    .main-container { 
      padding: 2rem; 
      height: 610px; 
      width: 415px;
    } 
  
    a { 
      font-size: 15px; 
      padding: 16px 0; 
    }
  }
  

  This would save code and simplify the use of media queries.

  hope this helps! 😁

  Marked as helpful

  0