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

my solution

Ahmed Alzarkaβ€’ 100

@Ahmedalzarka

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


any recommendations

Community feedback

P
visualdennisβ€’ 8,375

@visualdenniss

Posted

Hey there, your solution looks great!

You might also consider making it more responsive for mobile devices by adding a media query like @media (max-width: 500px) { .your-container { flex-direction: column; ...rest of the container code} } and avoid using position absolute.

Hope you find this feedback helpful!

Marked as helpful

0

@0xabdul

Posted

Hey there ! πŸ‘‹ Congratulations you finished the Statspreview card component...πŸŽ‰

  • Some suggestions for you improve your codeπŸ€”

Html 🏷️:

  • for Accessibility reports using the non-semantic elements (or) semantic elements..
  • semantic elements : <aside> , <artical> , <main>, <header> ,<section><footer>, <form> ect..
  • non- semantic elements : <div> , <span> ect ...
  • for easy way to clear the Accessibility reports using non semantic elements Ex :
<div class="container" role="main">
//Whole html code wraping the div tag named "container"..πŸ“
</div>
  • Whenever using the img tag should be put the alt Attributeβœ… and mention the alt discription Ex : <img src="logo.jpg" alt="logo"></img>
  • I Hope it's useful for you and wating for your next project ❀️
  • Happy Coding πŸ˜ƒ

Marked as helpful

0
Kamilβ€’ 260

@Wuczek

Posted

Great job!

You didn't make use of image file image-header-mobile.jpg in images folder

I would recommend to change your <div class="image-container"> into :

<picture class="image-container">
  <source srcset="images/image-header-mobile.jpg" media="(max-width: 600px)" />
  <img src="images/image-header-desktop.jpg" alt="" />
</picture>

so that your image file automatically changes based on certain width, you can read more about <picture> tag here: <picture> tag

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