Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Proof Section with HTML, CSS & Flexbox

Paul Verdure• 55

@Paul-Verdure

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback is very much welcomed. Also, one specific question. Do you usually start by building the mobile version and then the desktop one, or the opposite?

Many thanks and nice day!

Community feedback

hardy• 3,660

@hardy333

Posted

Everything looks so clean... There are very few solutions like that , very nice ...

I sometimes use mobile first but, most of the time desktop first, because I am more comfortable with desktop first. in general mobile first have some advantages over desktop first but it is not that big deal as people say on youtube xD mobile first is not like you are done job in way more easy or optimized.

Marked as helpful

3

Paul Verdure• 55

@Paul-Verdure

Posted

@hardy333

Hi!

Thanks so much for your feedback. For a beginner like me, it is very motivating and encouraging.

Also, thanks for the advice. I try to take a look at each preview in order to see which conversion would be the most practical :).

1
Shahin NJ• 1,190

@SJ-Nosrat

Posted

Hi Paul, Nice work! I always start with the Mobile first approach as this is best practice; there's a really nice video by Kevin Powell which explains as to why?

Then you can use @media (min-width: /*your size here*/) to build out for larger screen sizes successively.

Also, please add the below code to your existing CSS styles;

html {
 height: 100%;
}

body {
 min-height: 100vh;
}

The above code will take up the full height of the available view-port height.

I've also taken a look at your accessibility report, I've explained the issues for some of the main ones, as follows:

  1. Your <img class="star" src="icon-star.svg"> element is missing the alt attribute.
    • In the above case I'd do this: <img aria-hidden="true" class="star" src="icon-star.svg" alt=""> since the stars are for decorative purposes.
  2. Document should have one main landmark : this is due to using div tags as wrapper; I'd suggest using semantic HTML, you can find the resource here.

Hope the above helps!

Keep on coding and best of luck with your coding journey!

Marked as helpful

0

Paul Verdure• 55

@Paul-Verdure

Posted

@shahin1987 hi and thanks so much for your feedback. I will include these remarks to my next projects :-)

0

Please log in to post a comment

Log in with GitHub
Discord logo

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