Skip to content
Submitted almost 3 years ago

Responsive Ecommerce Product Page With Lightbox

Corvida Raven•680
@SheGeeks
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...E-commerce product page
  • HTML
  • CSS
  • JS
3intermediate

Design comparison

SolutionDesign

Solution retrospective

I would love to learn about ways to reduce having so many JavaScript event handlers for all the interactions in this challenge and whether having so many can cause performance issues.

Any additional feedback is also appreciated and welcomed!

Community feedback

  • Ren Ras•420
    @renrasPosted almost 3 years ago

    You can look into using event delegation where you put an event listener only to the parent.

    Too much event listeners are only bad depending on the function inside it. Most of your functions run in O(n) time or less which is good.

    Marked as helpful
    1
    Corvida Raven•680
    @SheGeeksPosted almost 3 years ago

    @renras I will look into using event delegation for the quantity buttons and lightbox navigation icons. That will hopefully remove two or more listeners from my code.

    0

Please log in to post a comment

Log in with GitHub

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