[PATCH] add HikeBikeMap and ÖPNV-Karte to Jmapviewer d/l dialog

[cmuelle8]

nur ein paar Details, die einem Mapper das Leben vereinfachen..

Gruß,
cmuelle8

[cmuelle8]

add some more common tile sources to the builtin jmapviewer d/l dialog

[stoecker]

The Ö in ÖPNV should be coded using \u....

[bastiK]

Any tms imagery source that the user has selected will show up in the jmapviewer, so shouldn't we rather put it on http://josm.openstreetmap.de/wiki/Maps (default=true)? That way the user can get rid of these entries if they want to (for whatever reason).

[stoecker]

Added, but not as default, as they are not worldwide.

[cmuelle8]

I have not checked your new commit yet, but it seems JMapViewer is used at two places

1) as TMS-LayerBackground
2) in the Data-Download-View obtainable by Strg+D

My workflow, when fixing things, is

A) to first browse the rendered map in the Data-Download-View
B) then select the area, hit download - the following steps of my workflow don't necessarily need the TMS-Layer anymore (I often just use it in the Download-Dialog to find errors)

To me it seemed, that the TMSs available in the Download-View were not configurable by the TMS/WMS Tab in Prefs - this is why I orginally patched the code. Maybe the Download-View should be synchronized then to the TMSs the user set active on TMS/WMS Tab in Prefs!?

Greetings

[bastiK]

Replying to cmuelle8:

I have not checked your new commit yet, but it seems JMapViewer is used at two places

1) as TMS-LayerBackground
2) in the Data-Download-View obtainable by Strg+D

My workflow, when fixing things, is

A) to first browse the rendered map in the Data-Download-View
B) then select the area, hit download - the following steps of my workflow don't necessarily need the TMS-Layer anymore (I often just use it in the Download-Dialog to find errors)

To me it seemed, that the TMSs available in the Download-View were not configurable by the TMS/WMS Tab in Prefs - this is why I orginally patched the code. Maybe the Download-View should be synchronized then to the TMSs the user set active on TMS/WMS Tab in Prefs!?

It works already like that, see above comment.

[cmuelle8]

Replying to bastiK:

Replying to cmuelle8:

Oh, ok sorry for bothering, somehow I did not receive the comment by mail or it passed by unnoticed.. Thanks for the quick adoption. Have a nice weekend (if that's possible after japan news..), Christian