Responsive Four Features Cards

Community feedback

• 

  Aakash Verma•9,520

  @skyv26Posted 2 months ago

  Hi @AlizaIkram,

  • Great job! Your project is responsive and works perfectly even on larger screens. The layout structure is clean and user-friendly. 👍

  • A small suggestion: Since it's just a single section page with four cards, using the <section> and <article> tags may not be necessary. You could simplify the structure as it's a straightforward layout. For example, you could just use a <div> for the cards.

  <div class="cards">
      <div class="card card1">
          <h2>Supervisor</h2>
          <p>Monitors activity to identify
      </div>
  </div>
  

  • Regarding the float property for the SVG, while it works fine, it’s a bit outdated. Instead, you could update the CSS to use flexbox for better alignment and maintain modern practices:

  .card {
      display: flex;
      flex-direction: column;
      gap: 10px;
  }
  
  svg {
      align-self: flex-end;
  }
  

  This will provide a more flexible and maintainable layout. 😊

  Keep up the great work! If you need any help, feel free to ask!

  Marked as helpful

  1

  ---

  

  Aliza Ikram•100

  @AlizaIkramPosted 2 months ago

  @skyv26 Thank you so much for clearing out these things! Also, I'm so happy to see your comprehensive feedback. Really appreciate that!

  1

  

  Aakash Verma•9,520

  @skyv26Posted 2 months ago

  @AlizaIkram You're welcome. It's my pleasure to help people because doing this also increases my knowledge and understand the other's perspective.

  1