Remove the catchall from #roads-text-name #1

Closed
gravitystorm opened this Issue Dec 2, 2012 · 3 comments

Comments

Projects
None yet
2 participants
@gravitystorm
Owner

gravitystorm commented Dec 2, 2012

There's a catchall at the end of #roads-text-name that draws labels on all kinds of things. This should be reworked into explicit rules.

At the moment it'll label all kinds of linear features, due to the rather bizarre sql query. It should only render roads, and if other things (underground power cables?) need labelling, there should be explicit rules for these to allow people to turn them on and off as they see fit.

Also, the catch-all leads to needing to explicitly override the values (e.g. halo-radius: 0;) on the more explicit rules.

windu2b referenced this issue in windu2b/osmfr-cartocss Apr 1, 2013

Merge pull request #1 from windu2b/master
 Ajout du logo Twisto, sur le réseau de l'agglomération caennaise
@skorasaurus

This comment has been minimized.

Show comment Hide comment
@skorasaurus

skorasaurus Apr 23, 2013

@gravitystorm

This will be in 2.0, not 1.0, right ?

@gravitystorm

This will be in 2.0, not 1.0, right ?

@gravitystorm

This comment has been minimized.

Show comment Hide comment
@gravitystorm

gravitystorm Apr 23, 2013

Owner

We're already at 2.2.0, so this is in scope as part of the "Easier to wrangle" aspect - it's currently very hard to customise the map if you want to turn off some of the (annoying) text labels.

Owner

gravitystorm commented Apr 23, 2013

We're already at 2.2.0, so this is in scope as part of the "Easier to wrangle" aspect - it's currently very hard to customise the map if you want to turn off some of the (annoying) text labels.

@gravitystorm

This comment has been minimized.

Show comment Hide comment
@gravitystorm

gravitystorm Feb 21, 2014

Owner

Fixed in 286d098

Owner

gravitystorm commented Feb 21, 2014

Fixed in 286d098

gravitystorm pushed a commit that referenced this issue Sep 24, 2014

pnorman pushed a commit that referenced this issue Nov 23, 2015

Merge pull request #1 from pnorman/admin_planning
Add additional SQL condition for very low admin logic

kocio-pl added a commit that referenced this issue Apr 16, 2018

Adding rendering for shop=video_games (#3167)
* added rendering for shop=video games (#1)

* Update amenity-points.mss

* Update project.mml

* Create video_games.svg

* Update video_games.svg

* added line breaks for better readability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment