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

Less dominant construction color #2292

Merged
merged 1 commit into from
Aug 22, 2016

Conversation

kocio-pl
Copy link
Collaborator

@kocio-pl kocio-pl commented Aug 19, 2016

Resolves #1197.

As originally proposed by @daganzdaanda, made 10% lighter with 5% desaturation. Lightening stops hinting that construction areas are more important than buildings and desaturation takes away the golden tint, which is currently overused in osm-carto, making also construction area more distinct than lightened-only version.

  1. City landscape, comparison with bigger buildings
    Before
    ldlg22vf

After
4ejz73xi

  1. City detail, comparison with buildings, trees and special buildings (tagged as place of worship)
    Before
    dp7zxvik

After
fdegunzf

  1. City detail, comparison with buildings and landuse=garages
    cvh2xwnv

  2. City detail, buildings on construction area
    u2erjogr

@EdLoach
Copy link

EdLoach commented Aug 19, 2016

"Lightening stops hinting that construction areas are more important than buildings" - except that construction areas are more important than buildings, as they indicate to mappers that they need regular re-surveying to check on progress. Of course these days they are usually supplemented by notes with comments used to give progress updates, so how they look on the tiles is less important than it was. I have no real opinion on how they should look though.

@kocio-pl
Copy link
Collaborator Author

General map should be careful with what it shows as important. In my opinion it should rather explain the space than deal with searching things. Mappers should use data-related tools (like Overpass query or special layer) to not overlook smaller construction sites.

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.

3 participants