-
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
Revert white color for unimportant roads #2769
Revert white color for unimportant roads #2769
Conversation
I prefer the white. |
We might need to re-add https://github.com/gravitystorm/openstreetmap-carto/pull/2654/files#diff-f61408bc1ffb6636dbaf2bf9e38a4177L1043 and further too. |
@meased I think there will be lot of discussions with other changes, so we can change this color later. @math1985 Do you mean this block only (till https://github.com/gravitystorm/openstreetmap-carto/pull/2654/files#diff-f61408bc1ffb6636dbaf2bf9e38a4177L1068) or something more? Altering road code is quite a challenge nowadays. |
I think so, but make sure to test it anyway. Road width might need changing too. |
e3a8e42
to
bafcd49
Compare
I'm not sure how to test it - with reverted I'm not even sure if anything more than color change was the problem to solve. |
roads.mss
Outdated
@@ -1048,30 +1048,32 @@ tertiary is rendered from z10 and is not included in osm_planet_roads. */ | |||
.tunnels-fill[zoom >= 10] { | |||
|
|||
::halo { | |||
[feature = 'highway_motorway'][link != 'yes'][zoom >= 8][zoom < 12], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This block should be kept (it is the halo for low-zoom roads). So we need both blocks now.
bafcd49
to
edc04e9
Compare
edc04e9
to
6815387
Compare
Done and it works, anything else that should be done here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Resolves #2731.
Reverts unimportant roads color from white to @residential-casing color (as in v4.1.0).
Before revert:
After revert: