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

Refine rendering of campsites #2770

Closed
wants to merge 2 commits into from
Closed

Conversation

giggls
Copy link

@giggls giggls commented Aug 23, 2017

I'm unsure if something like this should be discussed as an issue first. It comes in the form of a pull request because the feature it is already active in German style.

The Wiki proposes refined rendering of campsites for years now.

Here is an implementation:

  • Render camping/caravaning icon for campsites with tags tent=yes and caravans=yes
    test-camp-caravan
  • Render unchanged in case of caravans=no or unset
    test-campsite
  • Render backcountry icon if tagged backcountry=yes
    test-backcountry

To me there are two known issues:

  • The blue color does not look very nice. This is why I changed accomondation color to amenity brown in German style.
  • The backcountry icon could be more intuitive. I don't know if the trees are recognized as such. I'm open to other suggestions. I already experimented with campfire icons which where even less noticeable.

* Camping/Caravaning
* Backcountry
* Campsite without specific tags (as up to now)
@imagico
Copy link
Collaborator

imagico commented Aug 23, 2017

In general if you want to suggest rendering new features it is good practice to do some research into whether these tags are well established and consistently used and documented. This looks rather positive in this case though.

Large icons are a problem though, current policy says they should be 14x14 pixel. You could try if the tent symbol is still recognizable at a smaller size.

With the backcountry icon you could try showing a hiker instead of trees - this would also be less climate specific.

@matthijsmelissen
Copy link
Collaborator

I think this rendering is too specific for a general purpose map.

@giggls
Copy link
Author

giggls commented Aug 23, 2017

While this might be the true in the backcountry case it is shurely not in camping and/or caravaning. Even road signs differ between them.

@imagico: 14x14 is the size of the original tent symbol. Making it smaller will make it more or less irrecognizable. Campgrounds are usually fairly large objects Thus the slightly bigger size of the rendering will IMO look better that stying at 14x14.

Copy link
Collaborator

@pnorman pnorman left a comment

Choose a reason for hiding this comment

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

The icon size is an issue. Other cases where an icon is made of a symbol used elsewhere + something else (e.g. bike parking) are still 14x14

@kocio-pl
Copy link
Collaborator

kocio-pl commented Sep 9, 2017

@giggls Are you willing to try with smaller icons? I think that if they have a common visual thread (a tent) it wouldn't be a problem to recognize them as kind of campsite, no matter if we know what exactly type it is, so a size is the only thing that keeps me from approving this PR.

@giggls
Copy link
Author

giggls commented Sep 11, 2017

I already tried using standard sized icons but discarded the idea. . Especially the combined tent+caravan icon is barely recognizable. It might work with the tent+trees one.
IMO the standard size campsite icon is already very small anyway compared to the size of the physical object they usually represent.

@kocio-pl
Copy link
Collaborator

IMO the standard size campsite icon is already very small anyway compared to the size of the physical object they usually represent.

But that's true with every icon - see for example airport. The area however is a different thing - maybe we lack borders.

I think caravan icon is not really designed (it doesn't look consistent), but backcountry icon is more promising for me. You could squeeze it probably - one tree could be enough, I guess.

@kocio-pl
Copy link
Collaborator

It looks like the icon redesign is not going to occur soon, so I close this ticket, but a new one can be made once we have more compact icons. Thanks anyway!

@kocio-pl kocio-pl closed this Sep 22, 2017
@giggls giggls deleted the upstream branch May 18, 2018 09:09
@Adamant36
Copy link
Contributor

Adamant36 commented Dec 23, 2018

@Tomasz-W, this is one I think would be cool and useful to see. Especially for backcountry. Any chance you could come up with icons that fit the 14x14 standard? I imgine backcountry could possibly be doable if not the other one.

Or we could use icon side by side rendering maybe.

@giggls
Copy link
Author

giggls commented Dec 23, 2018

@Adamant36 I first thought about multiple icon rendering, but did not succeed in doing this using carto.

@Adamant36
Copy link
Contributor

Adamant36 commented Dec 23, 2018

@giggls, its doable. @jeisenbe had a PR (or maybe an issue) a while back for springs that used it. I'll try to find it or maybe he can enlighten us as to how its done. If I remember correctly it got rejected in that instance, but maybe it would work in this one. Its worth a try if 14x14 icons can't be done.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Dec 23, 2018 via email

@Adamant36
Copy link
Contributor

@jeisenbe, your probably right. I don't see why we couldn't fit the information in a 14x14 canvas somehow. Thinks for the extra information. I knew it was something like that.

@Tomasz-W
Copy link

Sorry, I can't design a readable 14x14 icons with such elements and details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants