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 (A full zoom-out on combned Brush and Zoom chart results in incorrect brush selection being drawn #2042, Victory Brush/Zoom out of sync #2023)
• Issues with setting the zoom domain (Setting state in createContainer combination of zoom with either brush or cursor causes issues #1660)
• Alignment issues (Cursor container misalignment with createContainer("zoom", "cursor") #1451)
• Issues with time scale (BrushContainer doesn't work properly with scale='time' #1560)
• Panning inconsistencies (Pan chart without redrawing everything #2029, Horizontal charts do not pan at the correct rate  #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.
#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 #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.