-
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
White mid-zoom roads do not work on some backgrounds #2731
Comments
I noticed the same when testing the bright roads without landcover fading and found that adding a darker halo with |
I'm not sure if that's a bad thing or even regression. Midzoom changes are about intentionally hiding some details (and at this zoom level I wouldn't like them to be more visible): |
If the visibility was consistent I'd see it as okay, but it's not consistent. |
For what it's worth, I agree with @pnorman that there is an issue here. |
What do you plan to do with it? Do you want to make any changes before v4.2.0? |
I also agree that issue exists. |
I'm quite busy with other stuff at the moment, so unfortunately I won't have time to work on it the next couple of weeks. I don't think we should hold up v4.2.0 because of me. I'm fine with reverting this change to the old color, or perhaps somebody has a better solution. Rolling it out as is and improving it in 4.3.0 is also an option to me. |
I would probably revert this particular change as the safest choice for now. I'm pretty sure we will tune midzoom in the nearest future anyway, because it's biggest visual change from the new road system introduction 2 years ago, so there will be good opportunity to think what we could do with these roads. |
In case someone wants to try it - the darker halo with I have not tried it with the faded landcovers or in urban areas with largely unmapped landcover. |
With the changes in 5fd79b7, there is a huge difference in how the roads look between different areas
In Surrey, the roads are readable with the residential background. In Langley, they're not, with most of them against our base colour.
The text was updated successfully, but these errors were encountered: