View Issue Details

Related ChangesetsJump to NotesJump to History
This bug affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
10589Bug reportsPluginspublic2016-02-25 16:012016-05-03 11:22
Reporterjlpoole Assigned Totpartner  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.50.x 
Summary10589: OSM - Aerial View Limited To Specific Zoom Level 10
Description

When using the Open Street Map ("OSM") widget, there appears at the top right an icon which lets the user select three modes: a) Street Map, b) Aerial Imagery, and c) Imagery with Streets.

If I set the default zoom level to 17 for a map of Napa, California, the street map appears. If in zoom level 17 (or really any zoom higher than 10) I select Aerial Imagery, the map goes to grey. If I then select the minus sign, the map zooms out to level 10 and then the zoom buttons have the plus "+" character greyed out. I'm guessing the code then recognizes that zoom level 10 is the maximum zoom when display the images for "Aerial" -- using a different tile server.

I'm also guessing that the image layers are limited zoom resolution 10. An unsophisticated end user seeing the choices available to him/her could select the Aerial Imagery and only see grey at zoom levels higher than 10 and then conclude that the tool is broken.

Ideally, the Aerial feature should not be available at a higher level, or there should be a warning that the higher zoom level for that particular tile server is not supported.

Steps To Reproduce

create an short text question using the OSM map provider, set the zoom level higher than 10

TagsNo tags attached.
Attached Files
limesurvey_survey_984767.lss (12,400 bytes)
Bug heat4
Complete LimeSurvey version number (& build) 88f410ed998361dfe63fbced8ec
I will donate to the project if issue is resolvedNo
BrowserFirefox 44.0.2
Database type & versionPostgreSQL 9.4.5-r2
Server OS (if known)Gentoo Linux 4.1.12-gentoo
Webserver software & version (if known)Apache 2.2.31
PHP Version 5.6.18

Users monitoring this issue

There are no users monitoring this issue.

Activities

c_schmitz

c_schmitz

2016-02-26 13:53

administrator   ~35739

Last edited: 2016-02-26 13:53

Tony, can you please have a look at this?
We need some JS code in map.js that hooks into the layer change event and zooms to maxlevel of the layer if it is > maxlevel.
tpartner

tpartner

2016-03-02 22:29

partner   ~35853

Carsten, I can find no way of determining a maxZoom level for a layer in a given world location. The tile images are inserted regardless of zoom level, just sometimes they are simply the no-image icon which, as in GMaps, indicates that there is no image for that resolution.

So, I think we have two choices here...

1) We allow zooming in the Aerial Imagery layer up to 19 - the same level as the Street map (for some reason it is currently restricted to 10). In this case, depending on where in the world you are, at zoom levels somewhere between 12 and 19 when switching to Aerial or Hybrid you will see the no-image icon and the zoom control will behave normally. I believe that this is how Leaflet intended it to behave and is what I recommend.

2) On switching to Aerial or Hybrid views we can force the map to zoom level 11 which should show actual aerial image tiles for most of the world.

Thoughts?
c_schmitz

c_schmitz

2016-04-04 16:31

administrator   ~36986

I would prefer option 2 - can you make the change, please?
tpartner

tpartner

2016-05-02 18:48

partner   ~38128

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=18858
c_schmitz

c_schmitz

2016-05-03 11:22

administrator   ~38137

Version 2.50 Build 160503 released

Related Changesets

LimeSurvey: master b5ad3fad

2016-05-02 16:48:52

tpartner

Details Diff 		Fixed issue 10589: OSM - Aerial View Limited To Specific Zoom Level 10
Dev: Zoom to 11 when switching to Aerial or Hybrid views		 Affected Issues
10589
mod - scripts/map.js Diff File

Issue History

Date Modified Username Field Change
2016-02-25 16:01 jlpoole New Issue
2016-02-25 16:01 jlpoole File Added: limesurvey_survey_984767.lss
2016-02-26 13:31 c_schmitz Assigned To => c_schmitz
2016-02-26 13:31 c_schmitz Status new => assigned
2016-02-26 13:52 c_schmitz Assigned To c_schmitz => tpartner
2016-02-26 13:53 c_schmitz Note Added: 35739
2016-02-26 13:53 c_schmitz Severity feature => minor
2016-02-26 13:53 c_schmitz Note Edited: 35739
2016-03-02 22:29 tpartner Assigned To tpartner => c_schmitz
2016-03-02 22:29 tpartner Note Added: 35853
2016-03-02 22:30 tpartner Status assigned => feedback
2016-04-04 16:31 c_schmitz Note Added: 36986
2016-04-04 16:31 c_schmitz Assigned To c_schmitz => tpartner
2016-04-04 16:31 c_schmitz Status feedback => assigned
2016-05-02 18:48 tpartner Changeset attached => LimeSurvey master b5ad3fad
2016-05-02 18:48 tpartner Note Added: 38128
2016-05-02 18:48 tpartner Resolution open => fixed
2016-05-02 18:50 tpartner Status assigned => resolved
2016-05-03 11:22 c_schmitz Note Added: 38137
2016-05-03 11:22 c_schmitz Status resolved => closed