doubleClickMaxStops does not remove zoom on double click

[AHTOHO4]

Describe the bug

You have a cool project, but I found unexpected behavior.

The problem is that I need to disable zoom on double click, leaving the behavior for UI buttons, but when I set doubleClickMaxStops: 0, then for some reason the zoom still remains

Expected behavior

When I stop doubleClickMaxStops: 0

I expect the maximum to be 0 times the zoom happens when I doubleclick.

How to reproduce

    <Lightbox
        open={isOpen}
        close={handleCloseLightbox}
        slides={attachments}
        index={currentAttachment}
        plugins={[Counter, Zoom, Fullscreen]}
        zoom={{ maxZoomPixelRatio: 2, doubleClickMaxStops: 0 }}
        render={{ slide: renderVideoSlide }}
    />

Screenshots / Logs

No response

Additional context

Interestingly, doubleClickDelay: 0.1 solves this problem, but in my opinion it is a bad solution.

[igordanchenko]

Thank you for reaching out. I'm afraid the doubleClickMaxStops prop does not provide the functionality you expect. This prop defines how many times you need to double-click or double-tap to go from the initial image to the maximum zoom level. Its value should typically be a positive integer, but a fractional value greater than or equal to 1 may also be used.

Zoom plugin provides several ways to control the zoom level:

• zoom buttons
• keyboard shortcuts
• double-click
• double-tap
• mouse wheel

There are currently no configuration options to disable any of the above controls.

Can you describe the issue you are trying to solve by disabling the double-click zoom-in control?

[AHTOHO4]

Yes, sure.

I have a video in a lightbox.
I made logic to play and pause when clicking on the video itself. Sometimes I run into a problem when I click play and then pause - it zooms in.

So I wanted to disable zooming on double click, but keep it in the control buttons.

[igordanchenko]

Thank you for the clarification. Please see if you can intercept onPointerDown event on your video player and disable its propagation:

onPointerDown={(event) => event.stopPropagation()}

[AHTOHO4]

Thanks for the solution, it will work, but it will bring new problems:

1. It will not be possible to change the slide by holding the mouse button on the video itself
2. It will not be possible to move the video when scaling

[igordanchenko]

Could you share a minimal example to help me better understand the UX you're building? The reason I ask is that the default Video plugin doesn't support zoom functionality on video slides, so I'm unsure how you envision the zoom feature working with them.