Refactor and fix brush and zoom behavior

[becca-bailey]

We have seen lots of issues come up around brush and zoom behavior, and the brush and zoom becoming out of sync in certain situations. Since it has been difficult to address this issues, VictoryZoomContainer is a good candidate for a refactor.

Let's consolidate some of the discussion around VictoryBrush and VictoryZoomContainer here in order to avoid duplicate issues.

Current issues:

• Incorrect selections in the brush container when zooming in and out (#2042, #2023)
• Issues with setting the zoom domain (#1660)
• Alignment issues (#1451)
• Issues with time scale (#1560)
• Panning inconsistencies (#2029, #2083)

This issue most likely depends on the state management milestone.

[siulca]

Probably should be looking into this one too since this is being refactored. https://github.com/FormidableLabs/victory/issues/2340

[micmcg]

Any update on the Zoom and Brush refactor @becca-bailey? Thanks

[Parker-Stafford]

Also curious about any updates here @becca-bailey, experiencing the exact same issue as described here https://github.com/FormidableLabs/victory/issues/1541, brushDomain is fully controlled, upon programmatically setting it, the brush does not re-render. The brushDomain is properly updating but x, y, and width values in BrushHandleComponent and BrushComponent are not updating.

[carbonrobot]

This issue scope has expanded greatly, but all the PRs appeared to be merged. I'm going to close this one and we can address any remaining problems in separate issues.