Four Card Feature Section

Community feedback

• 

  Sarah Hassan•145

  @sarah27hPosted about 5 years ago

  Great job KristijanJ, your design is responsive, I just recommend to use <header> element as you used <footer> for <div class="section-header">

  <header class="section-header">
          <h2>Reliable, efficient delivery</h2>
          <h2 class="bold">Powered by Technology</h2>
          <p>
            Our Artificial Intelligence powered tools use millions of project data
            points to ensure that your project is successful
          </p>
        </header >
  

  and use a font-size 12px a least for the body to be more readable

  2

  ---

  

  Sarah Hassan•145

  @sarah27hPosted about 5 years ago

  @KristijanJ You are welcome :) I want to mention that in style-guide recommend to Font size: 15px. So I have set it to Monitors activity to identify project roadblocks and for the cards, I set it to 13px. And you can get more useful info to use rem instead of px from AlexBueckig reply

  https://www.frontendmentor.io/solutions/four-card-feature-section-solution-zTsllYyx

  0

  

  Kristijan Jovanovski•30

  @KristijanJPosted about 5 years ago

  @sarah27h Thanks! I added a <header> element and increased some of the text which was 11px to 12px. Cheers!

  0
• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 5 years ago

  Awesome work on this challenge Kristijan and congrats on submitting your first solution! One small recommendation I'd have would be to change the .card-title elements from div elements to headings. Seeing as your main heading is h2 for this component, h3 would make sense.

  Let me know if you have any questions!

  0