correctly throttle victory-brush-container and add trailing false

src/components/containers/brush-helpers.js

@@ -1,6 +1,6 @@
 import { Selection} from "victory-core";
 import { assign, throttle, isFunction, isEqual, defaults } from "lodash";
-
+import { attachId } from "../../helpers/event-handlers.js";
 
 const Helpers = {
   withinBounds(point, bounds, padding) {
@@ -260,5 +260,9 @@ export default {
   onMouseDown: Helpers.onMouseDown.bind(Helpers),
   onMouseUp: Helpers.onMouseUp.bind(Helpers),
   onMouseLeave: Helpers.onMouseLeave.bind(Helpers),
-  onMouseMove: throttle(Helpers.onMouseMove.bind(Helpers), 16, {leading: true})
+  onMouseMove: throttle(
+    attachId(Helpers.onMouseMove.bind(Helpers)),
+    16,
+    {leading: true, trailing: false}
+  )
 };

src/components/containers/victory-brush-container.js

@@ -40,19 +40,20 @@ export default class VictoryBrushContainer extends VictoryContainer {
     target: "parent",
     eventHandlers: {
       onMouseDown: (evt, targetProps) => {
-        BrushHelpers.onMouseMove.cancel();
         return BrushHelpers.onMouseDown(evt, targetProps);
       },
       onMouseMove: (evt, targetProps) => {
-        evt.persist();
-        return BrushHelpers.onMouseMove(evt, targetProps);
+        const mutations = BrushHelpers.onMouseMove(evt, targetProps);
+
+        if (mutations.id !== this.mouseMoveMutationId) { // eslint-disable-line
+          this.mouseMoveMutationId = mutations.id; // eslint-disable-line
+          return mutations.mutations;
+        }
       },
       onMouseUp: (evt, targetProps) => {
-        BrushHelpers.onMouseMove.cancel();
         return BrushHelpers.onMouseUp(evt, targetProps);
       },
       onMouseLeave: (evt, targetProps) => {
-        BrushHelpers.onMouseMove.cancel();
         return BrushHelpers.onMouseLeave(evt, targetProps);
       }
     }