-
Notifications
You must be signed in to change notification settings - Fork 819
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
Allotments pattern #2267
Allotments pattern #2267
Conversation
Seems nice to me, service roads should be visible now, tracks and pedestrian are OK too, but footways are even less visible than currently (sample from here): |
IMO footways are not any worse with this change but this depends of course on the background color and orientation in the old styling. In general footway visibility is a generic problem not specific to this base color, see also #1793 |
Have you possibly tried to reverse these two colors? I think the reversed colouring (beige/brown dots, yellowish background), may actually look better (and will almost certainly solve footpath visibility). |
Another idea: maybe space between the dots could be wider? That would make the pattern less busy. |
@imagico Let me know what you think of the comments above. If it doesn't work out I'm fine merging the current version. |
Since this is not intended to be a fundamental change of allotments rendering completely different colors are outside the scope here. If there is something wrong with the current rendering or the one proposed here in addition to #1994 that should probably be discussed separately. A coarser pattern would equally be a bigger change and similarly i see no good reason for that at the moment. Coarse patterns have their own specific issues, especially with fine grained geometries. If the pattern is considered too strong even though it is definitely weaker than the old one it could be reduced in contrast of course but i would need to see cases where it is actually too strong to consider that. |
Fine, I merged the PR. |
Thanks. |
As said i don't see that. Footway visibility is a problem in general, was one with the old allotments rendering and still is with the new one but there are plenty of situations where it is far worse (like on bare_rock). So the way to go here is IMO to improve footway rendering, not to tweak area rendering at the cost of map clarity in other aspects. I considered various options when designing this PR - including no pattern and a dark pattern. But this one seemed to be the most suitable.
I can see no artefacts in that sample - would need to explain. |
That would be the best for me. However I remember that this proved to be really hard (especially #1359) and we found no solution, so I doubt we can resolve it. If you have some hint, it would be useful.
I don't know how to explain, that's why I posted an image - if you don't see it, it's not a problem for you. I see annoying irregular pattern of light brown dots. |
Thanks for sharing. Could you also provide a code for those attempts (so I can test it with footpaths) or just render it with footpaths examples yourself? |
Variant 3, with the dark brown spots on the beige background, actually looks more "natural" to me than the original proposal. With its darker "pattern" over a lighter background, it is also more consistent with most other pattern usage in OpenStreetmap-carto, like the forest visible in the example images. So, as to @imagico 's wishes for "consistency" (#2270), it might actually be better |
This changes the allotments pattern from transparent lines to a orchard-like grid dot pattern.
Closes #1994.
At the same time it slightly brightens the allotments and farmyard base color bringing farmyard on the same level as industrial etc (which seems fine in combination with the lighter farmland now). This way buildings and other stuff on allotments areas are better visible.
Sample (from here):