`fit()` does not take `scaleExtent()` into account when changing scale

[obraud]

Describe the bug
Using fit() may actually zoom in or out more than initially allowed by scaleExtent() (e.g. [0.001, 1] may actually result in a final scale of 1.5 on a small chart).
When zooming in/out afterwards, the scale will actually bump back to max/min scale.

To Reproduce
Example repo with scaleExtent set to 1

• click Fit to the screen => chart is zoomed out
• zoom out => chart is zoomed in back to scale 1

Expected behavior
fit() should use scaleExtent() min/max values when setting scale, or maybe provide an option to match scaleExtent?

Desktop (please complete the following information):

• OS: Windows
• Browser: Firefox
• Version: 3.1.1

[obraud]

I've got possible solution by extending zoomTreeBounds as such:

zoomTreeBounds({ x0, x1, y0, y1, params = { animate: true, scale: true, onCompleted: () => { } } }) {
        const { centerG, svgWidth: w, svgHeight: h, svg, zoomBehavior, duration, lastTransform } = this.getChartState()
        let scaleVal = Math.min(8, 0.9 / Math.max((x1 - x0) / w, (y1 - y0) / h));
        // match scale extent
        scaleVal = Math.min(
            Math.max(this.scaleExtent()[0], scaleVal),
            this.scaleExtent()[1],
        );
        let identity = d3.zoomIdentity.translate(w / 2, h / 2)
        identity = identity.scale(params.scale ? scaleVal : lastTransform.k)

        identity = identity.translate(-(x0 + x1) / 2, -(y0 + y1) / 2);
        // Transition zoom wrapper component into specified bounds
        svg.transition().duration(params.animate ? duration : 0).call(zoomBehavior.transform, identity);
        centerG.transition().duration(params.animate ? duration : 0).attr('transform', 'translate(0,0)')
            .on('end', function () {
                if (params.onCompleted) {
                    params.onCompleted()
                }
            })
    }