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

Change rendering for natural=spring #325

Closed
matkoniecz opened this issue Feb 2, 2014 · 68 comments
Closed

Change rendering for natural=spring #325

matkoniecz opened this issue Feb 2, 2014 · 68 comments

Comments

@matkoniecz
Copy link
Contributor

http://www.openstreetmap.org/?mlat=49.75063&mlon=19.79288#map=19/49.75063/19.79288

  1. It looks like 5, not like S
  2. I am unsure whatever S = Spring is obvious even for native speakers

I suggest some more obvious icon - maybe something like https://commons.wikimedia.org/wiki/File:RiverIcon-Spring.svg ?

Also orienteering maps have a good symbol for springs:

beztytuu

from http://www.maprunner.co.uk/simon/mapsymbols.jpg

@javbw
Copy link

javbw commented Feb 3, 2014

That orienteering icon looks good. is there any documentation for what -carto needs for icons?

@matkoniecz
Copy link
Contributor Author

According to #326 display of icon for node is completely independent from data about way where it belongs, so orienteering type icon that depends on where waterway goes may be impossible to implement.

@gravitystorm
Copy link
Owner

As a first stage it could be improved by just being a filled markersymbolizer.

The problem with rotating icons is coming up with a sufficiently complex SQL statement. It's possible to do this though.

@javbw
Copy link

javbw commented Feb 4, 2014

If we have an omni-directional icon, or one always displayed vertically, we can still make one without using a letter. The USGS topo map symbols http://www.lib.utexas.edu/maps/usgs_ref/usgs5.jpg use a little round circle to indicate well, and and round circle with a tail to represent spring.

If we use a little fountain symbol (blue, water squirting up) to denote a spring, the stream way (if there is one) would take care of the directional part of it all .

Maybe something like this.

spring_drops

@javbw
Copy link

javbw commented Feb 6, 2014

here is the old spring S with slightly rounded corners to look more like an S

spring_s

@mrwojo
Copy link
Contributor

mrwojo commented Feb 7, 2014

This is a prototype for a directional query:

(select distinct on (p.way) p.*, degrees(ST_Azimuth(p.way, ST_PointN(l.way, 2))) as azimuth_degrees
    from planet_osm_point p
    join planet_osm_line l
    on ST_DWithin(p.way,l.way,0.1)
    where p."natural"='spring' and l.waterway='stream') as natural_spring

I'd go with an icon that doesn't need to be directional, because the above query is overkill with the waterway already providing a visible direction. I noticed that JOSM uses one icon for both springs and fountains.

@matthijsmelissen
Copy link
Collaborator

Example icon: http://osm-icons.org/images/c/c9/24021.svg

@xsuchy
Copy link

xsuchy commented Jun 30, 2014

I would suggest http://miroslav.suchy.cz/osm/spring.png
! spring

I used to see this icon in our old maps and I very liked that. And it is very similar to what @javbw mentioned. It is created from from blue dot, which represent the spring itself, and the curve is made from letter S (as spring) and it is symbol of the water which flows from the spring.

@matthijsmelissen matthijsmelissen added this to the New features milestone Aug 18, 2014
@zdila
Copy link

zdila commented Sep 20, 2014

Please use this icon http://goo.gl/ku7aWM

I am author of the icon and I am releasing it under public domain.

If you need any modification to that icon, feel free to do it or ask me to do it.

@pnorman
Copy link
Collaborator

pnorman commented Sep 22, 2014

The problem with rotating icons is coming up with a sufficiently complex SQL statement

Coming up with a sufficiently complex SQL statement is never a problem for me! :)

Please use this icon http://goo.gl/ku7aWM

I am author of the icon and I am releasing it under public domain.

If you need any modification to that icon, feel free to do it or ask me to do it.

How does that look rendered to 16x16?

@matthijsmelissen matthijsmelissen changed the title Spring needs a better icon Change rendering for natural=spring Sep 24, 2014
@zdila
Copy link

zdila commented Oct 6, 2014

I've uploaded new revision so that image fits to 16x16px page. Preview on green background: spring11

@Klumbumbus
Copy link

I noticed that JOSM uses one icon for both springs and fountains.

JOSM-latest uses now different icons for spring and fountain. See https://josm.openstreetmap.de/ticket/10597

@nebulon42
Copy link
Contributor

For nodes that are part of a waterway a simple circle would be sufficient.
Like here: http://www.openstreetmap.org/#map=17/48.18177/16.24524
bildschirmfoto vom 2015-01-03 23 22 42

Detached springs would look weird, though:
spring

While I like the icon of @zdila it does not look so good connected to a waterway if not rotated properly:
bildschirmfoto vom 2015-01-03 23 37 32

Or something like @javbw proposed?
bildschirmfoto vom 2015-01-03 23 50 25

Admittedly, this needs more work so that it doesn't look like a flower, but I like the last variant best so far.

We also have to make sure that the icon is different enough from fountains (see #705 for rendering request).

@kocio-pl
Copy link
Collaborator

It's quite clear that it's not a flower because it has a water color and is not symmetric. It's much better than current symbol, so I would be happy to replace it as soon as possible. I think I will also use your icon for the fountain, so there's no risk of confusing them.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Sep 7, 2015

What about something like this on z14:
spring-1

@kocio-pl
Copy link
Collaborator

Another draft, based on the Osmic fountain icon (IMO lack of symmetry doesn't work with fountain anyway) with marina-text color:
z14
spring-osmic-14
spring-wien-14
z19
spring-brunnstube-19

@kocio-pl
Copy link
Collaborator

The same icon with lighter water-text color (#6699cc, also used for spring labels already) in park, without and with name label:
z14
spring-bay-text-14
z19
spring-bay-text-19
z16
spring-bay-text-agricola-16
z19
spring-bay-text-agricola-19

And in the forest:
z14
spring-bay-text-lux-14
z16
spring-bay-text-lux-16
z19
spring-bay-text-lux-19

I would say the lighter version is better in general. We can try using halo around it, but I don't know how to do it - just adding:

    marker-line-color: #ffffff; 
    marker-line-width: 1;

is not working. Maybe the icon has to be prepared to show it?

@nebulon42
Copy link
Contributor

IMO clearly better than what we have currently. Lighter is better but I think it is a bit too light for forest.

@kocio-pl
Copy link
Collaborator

Great! Do you like to make the color a bit darker or try a halo (in this case I need some instructions how to make it)?

@nebulon42
Copy link
Contributor

marker-line-color and similar only work for arrow or ellipse marker types. You could either experiment with the color or try a halo. I still think that halos would be a good idea around all icons, but they were rejected for space reasons. So it might be strange if spring would be the only icon with one.

If you still want to try it, here is an example: https://github.com/gmgeo/osmic-josm-style/blob/master/icons/outdoor/fountain-18.svg Please note that the halo colour and opacity cannot be changed from within the stylesheet.

@kocio-pl
Copy link
Collaborator

It makes sense for me - dropping the fill will make it more abstract, while not being cryptic, language specific symbol as currently.

Additional note: context is always important, not only the graphic design. Properly mapped area will include stream flowing out of the spring. Cases like this:
z14 2

are most probably just not finished. Maybe there are sources that don't have the visible stream flowing out, but they should be rare (does anybody know such places?). For example here the missing stream has been added eventually, along with many other objects:
screenshot-2018-2-15 openstreetmap

Also the wells should generally be placed near some populated places, not in the middle of nowhere, where springs can be found. It's not a hard rule, but looking at the image above I don't think of wells or watering places, even without the streams attached.

@Penegal
Copy link
Contributor

Penegal commented Feb 19, 2018

An example of @imagico's test without the blue fill:
image
image

At first, I thought of increasing the ring's thickness, but it may be voluntarily thin. What do you think?

@kocio-pl
Copy link
Collaborator

Hard to notice spring without the stream here (only the name helps). I would rather use the same color as stream, so it could be closer to original @matkoniecz design.

@Penegal
Copy link
Contributor

Penegal commented Feb 20, 2018

Too thick a border, I assume. With a larger border using water color:
image
image

Still too difficult to see, I think; maybe the white outer of the old symbol could be of some use here, couldn't it? Or a darker blue?

@kocio-pl
Copy link
Collaborator

In this version it''s even harder to see the single spring. I guess one has to experiment with different options and maybe blue fill will be better than lack of it.

@polarbearing
Copy link
Contributor

The discussion here takes a lot of side roads. I just want to consider that the spring icon needs to be visible in a body of water. See also #1482 about springs as areas.

@Penegal
Copy link
Contributor

Penegal commented Feb 28, 2018 via email

@Penegal
Copy link
Contributor

Penegal commented Mar 9, 2018

How about this way, with a white outline similar to the label:
image

@kocio-pl
Copy link
Collaborator

kocio-pl commented Mar 25, 2018

Could you test it on different backgrounds and without labels? Also some example with a stream flowing off. I guess this might be good solution.

@Penegal
Copy link
Contributor

Penegal commented Mar 27, 2018

I'll try that when I will be able to do so.

@kocio-pl
Copy link
Collaborator

Maybe just publish the code you have used, so we could test it more.

@Penegal
Copy link
Contributor

Penegal commented Mar 28, 2018

Here it is: Penegal@3b1d034 Neither the SVG code nor the commit history are clean, and this variant should be applied to spring_small.svg, too.

@Penegal
Copy link
Contributor

Penegal commented Apr 23, 2018

Other tests:
image
image

I also tried using the blue shade of wetlands, like on French maps:
image
image

The second version seems too prominent to me, but what do you guys think?

@StyXman
Copy link
Contributor

StyXman commented Apr 23, 2018

Something in between? I know it would be a new shade of blue, but I think it's worth it.

@Penegal
Copy link
Contributor

Penegal commented Apr 23, 2018

Mmh… Could do it; examples using a fifty-fifty mix:
image
image
image

Is it me or is the color similar to the label one? Or is it the antialiasing?

@kocio-pl
Copy link
Collaborator

It's a bit too weak in the woods (especially without label), so it could be a bit stronger.

In my experience the color perception of the label depends on the halo - see #2845 (comment) for example.

@Penegal
Copy link
Contributor

Penegal commented Apr 23, 2018

With a stronger halo:

  • fifty-fifty mix:
    image
    image
  • standard water color:
    image
    image

The first version seems more efficient to me, but I can live with the second if we must limit the number of colors displayed.

@kocio-pl
Copy link
Collaborator

I also like the first version more.

@Penegal
Copy link
Contributor

Penegal commented Apr 23, 2018

Other opinions before I PR this?

Penegal added a commit to Penegal/openstreetmap-carto that referenced this issue Apr 23, 2018
Penegal added a commit to Penegal/openstreetmap-carto that referenced this issue Apr 23, 2018
Penegal added a commit to Penegal/openstreetmap-carto that referenced this issue Apr 23, 2018
Penegal added a commit to Penegal/openstreetmap-carto that referenced this issue Apr 23, 2018
Penegal added a commit to Penegal/openstreetmap-carto that referenced this issue Apr 23, 2018
@Penegal
Copy link
Contributor

Penegal commented Apr 23, 2018

Sorry for the noise, I had trouble cleaning my history.

@xsuchy
Copy link

xsuchy commented Apr 23, 2018

I like the version with the stronger halo more.

kocio-pl pushed a commit that referenced this issue Apr 30, 2018
* Refurbished spring icon (fix #325 using @imagico first tests)

* Use @imagico code to display springs below waterways

* Removed useless SVG code

* Remove St_Centroid(way) performance regression
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.