Cascadenik icon indicating copy to clipboard operation
Cascadenik copied to clipboard

display:none preprocessor

Open nvkelso opened this issue 13 years ago • 6 comments

For situations where there is CSS for a class, but the end result is NULL in terms of drawing anything (like line-opacity: 0; where there is no other symbology applied), the data layer should be turned off and the CSS skipped.

nvkelso avatar Oct 31 '11 23:10 nvkelso

so special-case 0 for width and opacity and shortcut the symbolizer creation? Any others that spring to mind?

rcoup avatar Nov 01 '11 00:11 rcoup

Seems like it should apply to all:

stroke-opacity polygon-opacity line-width outline-width inline-width outline-opacity inline-opacity point-width point-height line-pattern-width line-pattern-height shield-size

On Mon, Oct 31, 2011 at 17:22, Robert Coup < [email protected]>wrote:

so special-case 0 for width and opacity and shortcut the symbolizer creation? Any others that spring to mind?

Reply to this email directly or view it on GitHub: https://github.com/mapnik/Cascadenik/issues/19#issuecomment-2585343

nvkelso avatar Nov 01 '11 00:11 nvkelso

I've gotten display: none working, here's the test on the branch: https://github.com/mapnik/Cascadenik/blob/16afc57c/cascadenik/tests.py#L2065

The opposite of display: none is display: map, and it’s implied for all elements.

I can likely make lines of zero width and so one work as described above too.

migurski avatar May 05 '12 00:05 migurski

Awesome!

_n

On Fri, May 4, 2012 at 5:10 PM, migurski < [email protected]

wrote:

I've gotten display: none working, here's the test on the branch: https://github.com/mapnik/Cascadenik/blob/16afc57c/cascadenik/tests.py#L2065

The opposite of display: none is display: map, and it’s implied for all elements.

I can likely make lines of zero width and so one work as described above too.


Reply to this email directly or view it on GitHub: https://github.com/mapnik/Cascadenik/issues/19#issuecomment-5522779

nvkelso avatar May 05 '12 00:05 nvkelso

Is this closed now?

nvkelso avatar Jun 01 '12 04:06 nvkelso

The special-case 0 is still open.

migurski avatar Jun 01 '12 05:06 migurski