Is onBrushEnd broken for the Brush component?

[JoeVanGundy]

Is onBrushEnd broken for the Brush component?

The docs say: Callback invoked on mouse up when a Brush size is being updated.

Not sure if I’m misunderstanding what this is suppose to do, but I would expect this callback to only fire when the user lets go of a mouse drag. Instead, I’m seeing it fire every time the value of the brush changes.

I'm keeping track of the current start and end dates using useState.
Since onBrushEnd is being calling so many times, I'm setting the state like 50 times during one short drag and performance is tanking hard.

Thanks!

[williaster]

Yep this is definitely broken. This may have been introduced in a recent update here and is specific to when useWindowMoveEvents={true}.

Shouldn't be too hard to fix, I'll try to get something out today.

[williaster]

closed by #1286

[JoeVanGundy]

closed by #1286

@williaster Forgot to say thanks for fixing this so fast!!
Thanks!!