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

Product Preview Card solution

Tanguyโ€ข 140

@OignonFugace

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I am using the background-image: url("images/image-product-desktop.jpg"); css property for displaying the image both in desktop and mobile view. Is it considered bad practice since it causes accessibility issues ?

Thanks for the feedback !

Community feedback

@MelvinAguilar

Posted

Hi @OignonFugace ๐Ÿ‘‹, good job completing this challenge! ๐ŸŽ‰

I like this solution for the challenge. Here are a few suggestions I've made that you can consider in the future if you're looking to improve the solution further:

  • In this challenge, you should not use the background property to set the image because this image has semantic meaning. Use the CSS background property if the image is not part of the content.

  • You can use a <picture> tag when you need to change an image in different viewports. Using this tag will prevent the browser from loading both images, saving bandwidth and preventing you from utilizing a media query to modify the image.

Example:

<picture>
   <source media="(max-width: 750px)" srcset="./images/image-product-mobile.jpg">
   <img src="./images/image-product-desktop.jpg" alt="your_alt_text">
</picture>
  • You could use the <del> tag to display the old price:
<del class="prices-item">
   <span class="sr-only">Old price: </span>$169.99
</del>

Note that I added the <span> with the sr-only class to the del element, this will provide more information about what your old price is about.

The sr-only class is a class that you can add to hide content visually but is only visible to screen-readers.

More information here.

I hope those tips will help you.

Good job, and happy coding!

Marked as helpful

0

Tanguyโ€ข 140

@OignonFugace

Posted

Hi @MelvinAguilar,

Thank you for the tips, most of which I didn't know about! I'll give them a try.

Have a great day.

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