If there is only one marker it is useful to set zoom only for this case.

Patch will be attached.

CommentFileSizeAuthor
#9 3108045-fit-locations-min-zoom.patch7.47 KBchristianadamski
#2 3108045-2.geolocation.Add-setting-Reset-zoom-if-single-marker-to-Fit-location-plugin.patch4.57 KBsuper_romeo

Comments

super_romeo created an issue. See original summary.

super_romeo’s picture

super_romeo commented
Status: Active » Needs review
StatusFileSize
new 3108045-2.geolocation.Add-setting-Reset-zoom-if-single-marker-to-Fit-location-plugin.patch4.57 KB
christianadamski’s picture

christianadamski commented

Isnt the value thing missing from the schema?

super_romeo’s picture

super_romeo commented

I have added 2 new values to the scheme...

christianadamski’s picture

christianadamski commented 
settings:
                 reset_zoom: false
+                reset_zoom_if_single: false
             freeogeoip:

That's only one :)

super_romeo’s picture

super_romeo commented

You are right. My fault. It is needed to add another one.

christianadamski’s picture

christianadamski commented
Status: Needs review » Postponed (maintainer needs more info)

I have a question: wouldn't it be better to alter this to a general "minimum zoom level" on fitlocations? Instead of checking the number of markers?

I assume your motivation for this is, that maps are zoomed in too much?

super_romeo’s picture

super_romeo commented

I assume your motivation for this is, that maps are zoomed in too much?

Yes.

If there are multiple markers I want they just be fitted. And if single marker I want to choose zoom.

christianadamski’s picture

christianadamski commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 3108045-fit-locations-min-zoom.patch7.47 KB

This is my version. Please try. I think it does what you want...

christianadamski’s picture

christianadamski commented

Tell me if I can merge this please :)

  • ChristianAdamski authored 5a54a4e on 8.x-3.x 
    Issue #3108045 by super_romeo, ChristianAdamski: Add setting "Reset zoom...
christianadamski’s picture

christianadamski commented
Status: Needs review » Fixed
super_romeo’s picture

super_romeo commented

Sorry, I was late :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.