Regression in feature zoom for MapWidget

[Claude Paroz]

When I rewrote MapWidget for OpenLayers 3 in [2ebfda38e], the code to autozoom to the initial feature (if present) was changed from:

this.map.zoomToExtent(feat.geometry.getBounds());
if (this.options.geom_name === 'Point') {
    this.map.zoomTo(this.options.point_zoom);
}

to:

this.map.getView().fit(extent, this.map.getSize(), {maxZoom: this.options.default_zoom});

As a result, the zoom is often the default_zoom, which is mostly not adequate at all (not enough zoom). If we simply remove the maxZoom, the zoom is too high for points (hence the point_zoom in the initial version). I think instead of maxZoom, a minResolution (see ​https://openlayers.org/en/latest/apidoc/module-ol_View-View.html#fit) would be more appropriate.

We would probably need to reintroduce some point_zoom variable, as providing a hard-coded value is not an option.

[Mariusz Felisiak]

Using minResolution sounds reasonable.

[Claude Paroz]

​PR

I finally opted for a hard-coded value, as IMO a resolution of 1 should be accurate for most (all?) layer types.

[GitHub <noreply@…>]

In 08f30d1b:

Fixed #33637 -- Improved initial zoom level in MapWidget.