Skip to content
Submitted about 1 month ago

four-card-feature-section using grid

Rido Septiawan•100
@riddsep
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I can apply the layout according to the design, I want to improve on frame and sass

What challenges did you encounter, and how did you overcome them?

tantangan yang saya hadapi adalah mencoba menggunakan grid layout untuk card nya dimana saya sebelumnya hanya familiar dengan flex

What specific areas of your project would you like help with?

everyone, I want input from friends about best practice bem and sass, I don't like tailwind

Community feedback

  • Adriano•38,250
    @AdrianoEscarabotePosted about 1 month ago

    Hello Rido Septiawan, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

    I noticed that you use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2

    <h1 class="hero__header--thin">Reliable, efficient delivery</h1>
<h1 class="hero__header--bold">Powered by Technology</h1>

    The rest is spot on.

    Hope it’s helpful to you. 👍

    Marked as helpful
    0
    Rido Septiawan•100
    @riddsepPosted about 1 month ago

    Wow, I'm really happy with the advice you gave, I appreciate it. thanks @AdrianoEscarabote

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord