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 more barrier icons #3602

Merged
merged 1 commit into from
Jan 3, 2019
Merged

Add more barrier icons #3602

merged 1 commit into from
Jan 3, 2019

Conversation

Adamant36
Copy link
Contributor

@Adamant36 Adamant36 commented Dec 26, 2018

This PR adds more icons for barriers. It closes #846

Motorcycle barrier
https://www.openstreetmap.org/#map=19/53.20660/-1.41045
motorcycle barrier
Cycle barrier
https://www.openstreetmap.org/#map=19/53.22609/-1.41716
cycle barrier
Kissing gate
https://www.openstreetmap.org/#map=19/52.45566/-1.58009
kissing gate
Full height turnstile
https://www.openstreetmap.org/#map=19/52.19147/-1.48791
full-height turnstile
Turnstile
https://www.openstreetmap.org/#map=19/51.50322/-0.11377
turnstile

Lots of gratitude to everyone involved, especially @Tomasz-W for making the icons and keeping track of everything.

@matthijsmelissen
Copy link
Collaborator

Great to see these being added!

@Tomasz-W
Copy link

I'm not sure about kissing gate icon: it's bigger than simple gate icon while it's usually smaller than car gate in real life, another thing is that icons can't be rotated to show a proper side of an entrance. I consider reusing curent gate icon instad of dedicated one. What do you think?

@Adamant36 Can you test turnstiles in these locations?:
https://www.openstreetmap.org/node/4089563550
https://www.openstreetmap.org/node/4099981907
https://www.openstreetmap.org/node/4121149096

@Tomasz-W Tomasz-W mentioned this pull request Dec 27, 2018
26 tasks
@Adamant36
Copy link
Contributor Author

I dont know. I like the kissing gate icon. Even if its bigger. The normal gate icon is pretty tiny. So I think its more on that. Their different things anyway. Plus, the current gate tagging is also used for pedestrian walkway gates in front houses. Its not just exclusive to cars.

I'll test it. You want it done with the bollard dot or with an icon?

@Tomasz-W
Copy link

Tomasz-W commented Dec 27, 2018

I'll test it. You want it done with the bollard dot or with an icon?

Bollard dot. The thing is that turnstiles are often mapped very dense and underground, so it may be unreadable and it needs testing to decide should we render them or not.

@jragusa
Copy link
Contributor

jragusa commented Dec 27, 2018

Motorcycle barriers look like a pliers and the vertical shape is not intuitive. Why not using the same for both motorcycle and cycle barriers ?

@Adamant36
Copy link
Contributor Author

Google pictures of them. They dont look the same in real life. The motorcyle barrier looks pretty close to how it looks in real life.

@Adamant36
Copy link
Contributor Author

Adamant36 commented Dec 28, 2018

@Tomasz-W, here you go. I think all the locations look OK. Location 1 and 2 in particular should probably be tagged as lines anyway.
Location 1
z17
location 1 z17
z18
location 1 z18
z19
location 1 z19
Location 2
z17
location 2 z17
z18
location 2 z18
z19
location 2 z19
Location 3
z17
location 3 z17
z18
location 3 z18
z19
location 3 z19

@trigpoint
Copy link

trigpoint commented Dec 28, 2018

Google pictures of them. They dont look the same in real life. The motorcyle barrier looks pretty close to how it looks in real life.

I would agree, looking at that symbol on the map I would instantly know what it is without needing to look at that the map legend.

@turnsole80
Copy link

What are everyone's thoughts with regards to rendering barrier=entrance? Maybe we could render it the same way as entrance=yes?

@matthijsmelissen
Copy link
Collaborator

To be honest, cycle barrier is not intuitive to me either - but I don't know if we can come up with something better?

@Adamant36
Copy link
Contributor Author

It looks like the picture of the cycle barrier on the wiki from above at least. I think the fact that its tagged on a highway would help to. Theres only a small number of things it could be. Plus, I dont know how else the icon could look.

@Tomasz-W Tomasz-W mentioned this pull request Jan 1, 2019
19 tasks
@matthijsmelissen matthijsmelissen changed the title Add more barrier icons [WIP] Add more barrier icons Jan 2, 2019
@Adamant36
Copy link
Contributor Author

@matthijsmelissen, I wouldn't necessarily call this a WIP. There doesn't seem to be a next step in the testing process anywhere. Unless I'm missing something. Someone just saying they don't like an icon doesn't really count as needing more work in my opinion (especially if they don't provide an alternative and the vast majority of people are for it). So I think its ready for testing/merging by an admin.

@matthijsmelissen matthijsmelissen changed the title [WIP] Add more barrier icons Add more barrier icons Jan 3, 2019
@matthijsmelissen
Copy link
Collaborator

I wasn't sure if there's anything else you wanted to do with the suggestions given. Thanks for indicating it's ready, going to review it then!

@matthijsmelissen
Copy link
Collaborator

The motorcyle barrier looks pretty close to how it looks in real life.

I think it's a misconception that icons should look like the object they depict. The best icons often don't. Clothes shops don't look like t-shirts, night clubs don't look like music notes, and churches don't (necessarily) look like crosses.

@matthijsmelissen
Copy link
Collaborator

I'm going to merge this now. If anyone has ideas for better icons, feel free to submit icons.

@Adamant36 Just a small note, in the future please be careful not to introduce unnecessary whitespace at the end of lines.

@matthijsmelissen matthijsmelissen merged commit 6148df3 into gravitystorm:master Jan 3, 2019
@Adamant36
Copy link
Contributor Author

My bad. I thought I checked it. Thanks for merging it.

@Adamant36 Adamant36 deleted the barrier branch January 3, 2019 23:23
@Adamant36
Copy link
Contributor Author

Adamant36 commented Jan 4, 2019

I think it's a misconception that icons should look like the object they depict.

Does that mean the next time I don't like an icon that I can argue against it by saying it doesn't look abstract enough?

@matthijsmelissen
Copy link
Collaborator

Does that mean the next time I don't like an icon that I can argue against it by saying it doesn't look abstract enough?

No, but you cannot argue that something is a good icon just because it looks like the object its depicting.

@Adamant36
Copy link
Contributor Author

Hhhmmm, good distinction. I guess a bunch of almost photo realistic icons would probably look horrible, even if they are close to what they are representing.

@wilmaed
Copy link

wilmaed commented Jan 12, 2019

@kocio-pl
Copy link
Collaborator

Please open new ticket with this proposition.

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.

Add rendering for barrier points
8 participants