Florian Bischof

@jonkoops the Release notes looking good for me. We should not forget to add them in [CHANGELOG.md](https://github.com/Leaflet/Leaflet/blob/main/CHANGELOG.md) too

@mourner there should be no breaking change but > Expose ESM entrypoint with Leaflet global (https://github.com/Leaflet/Leaflet/pull/8329 by @jonkoops) can break something. On the otherside how many will users will use...

@jonkoops my 2 cents to the release notes: I would not preview `Embracing modern JavaScript.` for v2, because that change can cause that we need much longer then expected and...

@ache051 please add it as PR maybe it will be merged

With `NaN` for `minZoom` while TileLayer creation, the `maxZoom` results to be `NaN` too.. ``` L.tileLayer("", {minZoom: NaN, maxZoom: 18}) ```

@Malvoz Tested with the zoom controls (which should already work) but they are not triggered by hitting `space` Have I understand it correct, that this should be fixed with only...

Please provide more infos, I can't reproduce this and your sample tiles are requested from `localhost`. Use https://leafletjs.com/edit.html to create a example.

I only want to document this: We should **not** remove the existing css classes because they are used for many custom controls

@johnd0e I think with the `isNaN()` we have a good start for validation but I think also that the input validation of L.Point and other Classes should be a part...

Nice PR, maybe you can also add something what break the loop if a Group has already called `getBounds()`. ``` var fg = L.featureGroup(); var lg = L.featureGroup().addTo(fg); fg.addTo(lg); console.log(fg.getBounds());...