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

Adding icon for amenity=university and amenity=college [WIP] #2361

Closed
wants to merge 1 commit into from

Conversation

kocio-pl
Copy link
Collaborator

@kocio-pl kocio-pl commented Sep 21, 2016

Related to #120 (partial solution).

Code based on amenity=hospital, but using current university/college label color.

Area, z15
Before
x10fsg0u
After
7gqkxtaj

Area, z16
Before
fvpjydrj
After
fbp9hjni

Area, z17
Before
evlp4xpi
After
dksxozgx

Node, z16
Before
dkmzlijr
After
trxnlh7l

Node, z17
Before
ws6roiqe
After
wg3nt5xd

Node, z18
Before
ertddgfy
After
hywvs2qx

@Circeus
Copy link

Circeus commented Sep 23, 2016

PLEASE not. Having a scaling name for these is one of the best thing about them, and anything with an icon has a name that can't scale anymore according (IIRC) pixel_area. In fact, I personally wish we could reasonably get rid of the hospital icon so hospital names could scale too.

@matthijsmelissen
Copy link
Collaborator

anything with an icon has a name that can't scale anymore according (IIRC) pixel_area

That's not a technical limitation, so if we want scaling labels in combination with icons we can define rules that allow this. Currently, caravan sites already do have a scaling label in combination with an icon.

See also #1355. @Circeus, I would be very interested in hearing your feedback on that issue.

@dieterdreist
Copy link

sent from a phone

Il giorno 23 set 2016, alle ore 12:33, Matthijs Melissen [email protected] ha scritto:

That's not a technical limitation, so if we want scaling labels in combination with icons we can define rules that allow this. Currently, caravan sites already do have a scaling label in combination with an icon.

but it's a limitation designwise, either we scale the icon along with the text (would be a big change) or it will lead to strange looks (tiny icon compared to text).
Maybe having 2 or even three icon sizes could eventually look ok, it surely would increase code complexity (offsets etc) quite a bit

@kocio-pl kocio-pl changed the title Adding icon for amenity=university and amenity=college Adding icon for amenity=university and amenity=college [WIP] Sep 23, 2016
@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Sep 24, 2016

We should probably have a general rule for such cases - see for example similar problem with prison: #1022

UPDATE - also some other examples:
#1019
#1020
#1052
#2073

@RobJN
Copy link

RobJN commented Oct 12, 2016

Do you need an icon for a university given that the name will usually include "university" in it?

@polarbearing
Copy link
Contributor

@RobJN - is there any research that supports your assumption? OSM defines the use of the university tag for third level institutions, and I know many of them not carrying 'university' in their name. The MIT is probably the most prominent on the technology sector.

@matthijsmelissen
Copy link
Collaborator

Do you need an icon for a university given that the name will usually include "university" in it?

Not saying we would, but we could show the university icon on lower zoom levels, before the name can be displayed, to indicate the campus location inside the city.

@RobJN
Copy link

RobJN commented Oct 13, 2016

No evidence just my hunch for the UK. Just seems like there was a lot of
attempt to clean up the map render and now loads of icons are being added.
Just wondered whether this was directed by a strategy. I'm sure you agree
that there is a balance between usefulness of icons and clutter.

On 13 Oct 2016 9:04 a.m., "Matthijs Melissen" [email protected]
wrote:

Do you need an icon for a university given that the name will usually
include "university" in it?

Not saying we would, but we could show the university icon on lower zoom
levels, before the name can be displayed, to indicate the campus location
inside the city.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2361 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE02ciGYOK7RzXKvC1KUp4JONsL-tN2nks5qzeX_gaJpZM4KDUfb
.

@kocio-pl
Copy link
Collaborator Author

Can we reach some agreement with this PR? If not, I'm gonna close this PR without merging in a few days.

@polarbearing
Copy link
Contributor

In favour of merging.

@kdudzik
Copy link

kdudzik commented Nov 28, 2016

I don't like this change. Icons suggest nodes or small areas. We could enable this icon for nodes only but enabling it also for areas makes the campuses look worse IMHO.

@kocio-pl
Copy link
Collaborator Author

We could enable this icon for nodes only but enabling it also for areas makes the campuses look worse IMHO.

That would be inconsistent with how we deal with hospitals. Do you think that universities are different in that regard?

@matthijsmelissen
Copy link
Collaborator

What about first trying to render the name (in italics), and if it doesn't fit render the icon (without label)?

@polarbearing
Copy link
Contributor

math1985> name (in italics), and if it doesn't fit render the icon (without label)

hm is that consistent behaviour someone would expect? Would that mean that in lower zoom I see an icon, when I zoom in the name?

kdudzik> Icons suggest nodes or small areas.

First time I hear about such rule.

kdudzik> We could enable this icon for nodes only but enabling it also for areas makes the campuses look worse IMHO.

Why? Maybe you have a campus that is tagged incorrectly, i.e. amenity nodes/buildings within the campus amenity?

Nodes and areas should not be treated differently in rendering, often a node is a first step in mapping before it is expanded to an area.

@kdudzik
Copy link

kdudzik commented Nov 30, 2016

@kocio-pl I don't think hospitals are any different - they should be handled the same way, true.

@polarbearing I don't know any rule, that's just my subjective opinion. I'm not quite convinced to it, but I would prefer it to the current proposal.
Maybe @math1985's idea could work.
In my opinion a small icon with a small caption (or even without a caption) in the middle of a large area (which it represents) looks just weird.

@kocio-pl
Copy link
Collaborator Author

kocio-pl commented Dec 8, 2016

I think we should first rethink how do we render similar objects and then go back to this problem. Thanks for the feedback!

@Tomasz-W
Copy link

Tomasz-W commented Jul 2, 2018

Further discussion in #3284

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.

8 participants