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 Using Float

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


This is my solution for this challenge. any feedback? Thankyou.

Community feedback

mr-geones 30

@mr-geones

Posted

Nice work!

However the website is not responsive, but I'm sure you're aware of it ;)

The "icon-cart.svg" within the green button was not visible in my browser. Changing the path src="/images/icon-cart.svg" to src="images/icon-cart.svg" solved the issue.

I think you can optimize your code by removing excessive elements. See the example below.

BEFORE

<div class="product-price">
  <div class="discount-price">
    <p>$149.99</p>
  </div>
  <div class="actual-price">
    <p>$169.99</p>
  </div>
</div>

AFTER

<div class="product-price">
  <p class="discount-price">$149.99</p>
  <p class="actual-prices">$169.99</p>
</div>

I'm a newbie myself, so unfortunately I don't have more feedback to give :)

Keep up the good work!

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