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

Order Summary Component [ HTML5 + CSS3 ]

#accessibility

@0xabdulkhalid

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Mobile First 📱 > 🖥

  • Nothing to complicated, layout was built with CSS Grid and Flex.

All feedback is greatly appreciated!

TWEAKS 🚀

  • Smooth zoom-in transition while hovering the card

Community feedback

@AbdulrhmanSoliman

Posted

Hello @0xAbdulKhalid, congrats on completing this challenge and for your awesome solution.

I suggest you setting the effect of the hover (scale) to the <picture> HTML tag element, because the scale property should relate only with the image when hovering on it.

I hope you find it helpful.🎉

Happy Coding

0

@0xabdulkhalid

Posted

@AbdulrhmanSoliman Assalamu alaikum warahmatullahi wabarakatuh brother !

  • Thanks for providing suggestion by spending your valuable time in evaluating my code
  • If i set the hover to <picture>, then it will ran through issues
  • I recommend you to manipulate the dom by making changes via dev console and then provide your feedback
  • Because we want to first validate our review before posting, i hope you can understand what i'm saying
  • Jazhakallahu kairan Brother
0

@AbdulrhmanSoliman

Posted

@0xAbdulKhalid

Wa'alaikum assalamu warahmatullahi wabarakatuh

I found from your CSS file this code below. .product:hover > picture > img{ transform: scale(1.2); filter: brightness(1.1); z-index: -1; }

I suggest changing the .product class to the <picture> HTML tag because we want to make the picture to be scaled when hovering only on this picture.

So to fix this issue I suggest you to use this code below picture:hover > img{ transform: scale(1.2); filter: brightness(1.1); z-index: -1; }

Thank you @0xAbdulKhalid I hope you find it helpful.

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