Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Stats Preview Card - HTML and CSS. (FlexBox and Mobile First)

@AlyferJT

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi there, that was a good challenge to work with media querys, i really liked it. I think that was all great, but if you have some feedback, i would appreciate :D

Community feedback

Hassia Issah 50,670

@Hassiai

Posted

Replace <div class="preview"> with the main tag and <h2> with <h1> to make the content/page accessible. click here for more on web-accessibility and semantic html

Every html must have <h1> to make it accessible. Always begin the heading of the html with <h1> tag wrap the sub-heading of <h1> in <h2> tag, wrap the sub-heading of <h2> in <h3> this continues until <h6>, never skip a level of a heading.

For the color of the image, add the background-image with a background-size of cover, background-blend-mode: multiply and opacity:0,8 to .preview-imagebox

.preview-imagebox{
background-image: url();
background-size: cover;
background-color: hsl()
background-blend-mode: multiply;
opacity: 0.8;
}

Hope am helpful.

Well done for completing this challenge. HAPPY CODING

Marked as helpful

1

@AlyferJT

Posted

Get it, i'm going to study that now. Thank you for the feedback :D @Hassiai

1
Luka Glonti 3,440

@lack21

Posted

Great job 👍, but I have a recommendation!

  • Replace height: 100vh to min-height: 100vh in the body, the difference is that, when you set height: 100vh to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to set min-height: 100vh, like this in case content is overflowing container will adjust to it.

  • Remove width: 100% from the body, body is a block level element which by default takes full width available, so width: 100% does nothing here!

Marked as helpful

0

@AlyferJT

Posted

@lack21 Ohh get it, i was with that problem on body height in the mobile screen. That will be very helpfull. Thanks!!!

1

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