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

Correct rendering of highway=path in tunnels #384

Merged
merged 5 commits into from Mar 11, 2014

Conversation

matthijsmelissen
Copy link
Collaborator

  • Make background of path in tunnels wider, so that it does not
    disappear below the path itself
  • Line-opacity of background of path should only apply to regular
    roads, not to tunnels

This PR depends on #382.

- Do not hide secondary road bridges on z12
- Do not hide tertiary, residential, unclassified, and road bridges on z13

Conflicts:

	roads.mss
This collapses, per road type, definitions that are the same for normal
roads, tunnels and/or bridges.
In addition, it gives some instances clearer names.
This makes it more easy to maintain the code, and to detect inconsistent
definitions.

Changes in rendering:
* Render minor service roads on bridges and tunnels smaller and on higher
  zoomlevels only, just like minor service roads that are not on a
  bridge or tunnel. This solves gravitystorm#269 on Github and 3883 on trac.
* Restore bridges of bridleway, footway, cycleway, path, lightrail on
  z13.
- Make background of path in tunnels wider, so that it does not
  disappear below the path itself
- Line-opacity of background of path should only apply to regular
  roads, not to tunnels
gravitystorm added a commit that referenced this pull request Mar 11, 2014
Correct rendering of highway=path in tunnels
@gravitystorm gravitystorm merged commit 36c0c4b into gravitystorm:master Mar 11, 2014
@matthijsmelissen matthijsmelissen deleted the paths-in-tunnels branch April 10, 2014 22:41
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.

None yet

2 participants