Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add OPNVKarte #2670

Merged
merged 2 commits into from
Jul 8, 2020
Merged

Add OPNVKarte #2670

merged 2 commits into from
Jul 8, 2020

Conversation

pnorman
Copy link
Contributor

@pnorman pnorman commented Jun 26, 2020

The layer has been approved by the OWG, I just took longer than expected to add it.

image

There was some discussion with @gravitystorm about naming in IRC a few weeks back. I think this naming is consistent with what we've done with other layers, using the layer name rather than a description. We've used that for MapQuest, HOT, Transport, and close to that to OpenCycleMap.

@@ -2498,6 +2498,7 @@ en:
cycle_map: Cycle Map
transport_map: Transport Map
hot: Humanitarian
opnvkarte: ÖPNVKarte
Copy link
Member

@tomhughes tomhughes Jun 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be ÖPNVKarte?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't that what it is? I don't see a typo

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh you're quite right, it's just very hard to see in this font, at least without my glasses...

@gravitystorm
Copy link
Collaborator

. I think this naming is consistent with what we've done with other layers, using the layer name rather than a description. We've used that for MapQuest, HOT, Transport, and close to that to OpenCycleMap.

This is the exact opposite of what I was trying to say! I think the user interface should show what is useful to the user, i.e. what the topic of the map layer is, not whatever the codename of the project is or which organisation is behind it.

For example, the "Standard" layer shows a roughly standard map - it doesn't have a narrower topic than that. We don't call this layer "openstreetmap-carto" since that would be meaningless to most people. We (thankfully) no longer call it Mapnik. And if the project that implements this is changed, again, perhaps to openstreetmap-cartographic, then I would expect it still to be called "Standard" in the user interface.

The "Humanitarian" layer is another one - it shows a map with the topic of "Humanitarian", but not "HDM-CartoCSS" (the project name) nor HOT (the organisation behind it).

The "Transport Map" layer shows a Transport themed map. The project is called Thunderforest Transport, by Thunderforest, but we don't use either of those in the layer picker.

The "Cycle Map" layer is named like it is for historical reasons, and could be better named something like Cycling. But again, it's neither OpenCycleMap nor mentioning Thunderforest.

In summary, my view of what we should be doing showing the topic, not the implementation details, of each layer.

Topic (shown to user) Project Name (not shown) Hosting Organisation Notes
Standard openstreetmap-carto OSMF
Cycle Map OpenCycleMap Thunderforest Topic could be renamed to "Cycling"
Transport Map Thunderforest Transport Thunderforest Topic could be renamed to "Transport"
Humanitarian HDM-CartoCSS HOT

(For the avoidance of doubt, as well as being a maintainer here, I'm also the creator of two of the existing styles. I'm wearing my OpenStreetMap hat here, not my Thunderforest hat, and I'm not asking for the Thunderforest brand name, or the project names of either of these two styles, to be shown in the user interface - quite the opposite.)

As for the MapQuest layer, or the tiles@home layer, or the Mapnik label - yes, those existed, and I think they were poor choices of label. I think we should learn from our previous mistakes, and continue to use topics as the label shown to the user.

@tomhughes
Copy link
Member

Oh I completely misunderstood what you meant on IRC then as I thought you were meaning the the current cycle later should use the OpenCycleMap name, and was in fact planning to rename it as such.

The problem we have going forward is that we're going to have two cycling layers and two public transport layers, so we will need to have some way to distinguish them!

@simonpoole
Copy link
Contributor

simonpoole commented Jun 26, 2020

May I point out that we already have precedent on how to handle this, namely the routing provider selection.

So I would suggest a scheme roughly like

Topic - Provider

or

Topic (Provider)

it may be appropriate to exchange "Provider" for a layer name if the provider prefers it that way, I don't think we really care about that aspect.

So we could have

Standard
Cycling (Thunderforest)
Cycling (xyz)
Transport (Thunderforest)
Transport (ÖPNVKarte)
Miscellaneous (Humanitarian)

(I would argue that the label Humanitarian is demeaning towards the providers and creators of the other layers and shouldn't be used but that is a different topic)

@pnorman
Copy link
Contributor Author

pnorman commented Jun 27, 2020

I don't really have a strong opinion on this, so if the maintainers will let me know which is preferred I'll change it.

@tomhughes
Copy link
Member

This is what the layer switcher would look like with the names @simonpoole suggested:

Screenshot_2020-06-28 OpenStreetMap

So we will definitely need to change the font size or something if we go for such long names.

@simonpoole
Copy link
Contributor

simonpoole commented Jun 28, 2020

This is what the layer switcher would look like with the names @simonpoole suggested:
...

So we will definitely need to change the font size or something if we go for such long names.
...

Obviously there is potential for endless bike sheding. For example we could have the categories as headings of some kind (and not repeat them on the button), or a myrid of other ways of doing this. That IMHO somebody with some design chops should have a look at.

@pnorman
Copy link
Contributor Author

pnorman commented Jun 28, 2020

Could we split up the discussion on what to call this layer in this PR and any long-term renaming?

@tomhughes tomhughes merged commit 4cf3c3b into openstreetmap:master Jul 8, 2020
@pnorman pnorman deleted the opnvkarte branch July 8, 2020 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants