Skip to content
Submitted about 1 month ago

Four_card_feature_section

abhi123-rgb•80
@abhi123-rgb
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 have learnt lot about grid in this project. I am trying write clean code and organise the code which makes it professional way of writing the code.

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

Nothing more but learnt grid layouts to well structure and organise the components in the site.

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

I want to know the mistakes what I done and how can improve writing the code more efficiently and perfectly to make it professional.

Community feedback

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

    Hi abhi123-rgb, how’s everything? I think your project turned out great! However, I have some feedback that I think might be useful:

    You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader.

    You can read more in MDN.

    If you want to centralize your component, we can do this:

    body { 
    display: flex;
    min-height: 100vh;
    justify-content: center;
    align-items: center;
}

    The rest is amazing.

    I hope this is helpful. 👍

    0

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