mapnik / Cascadenik

Cascading Sheets Of Style for Mapnik
http://github.com/mapnik/Cascadenik/wiki/Cascadenik
BSD 3-Clause "New" or "Revised" License
111 stars 20 forks source link

text-max-char-angle-delta doesn't output max_char_angle_delta #25

Closed ejsarge closed 10 years ago

ejsarge commented 12 years ago

Targetting mapnik 0.7.1

mss

#course-trail-label trail_name
{
    text-spacing: 256;
    text-placement: line;
    text-max-char-angle-delta: 37;

    text-face-name: 'PT Sans Narrow Regular';
    text-fill: #5a3b00;
    text-size: 8;

    text-halo-fill: #f7f7e6;
    text-halo-radius: 1;

    text-dy: 10;
}

Actual resulting xml snippet:

   <Rule name="rule 4">
      <TextSymbolizer dy="10" face_name="PT Sans Narrow Regular" fill="rgb(90,59,0)" halo_fill="rgb(247,247,230)" halo_radius="1" name="trail_name" placement="line" size="8" spacing="256" />
    </Rule>'

Expected resulting xml snippet:

<Rule name="rule 4">
    <TextSymbolizer dy="10" face_name="PT Sans Narrow Regular" fill="rgb(90,59,0)" halo_fill="rgb(247,247,230)" halo_radius="1" name="trail_name" placement="line" size="8" spacing="256" max_char_angle_delta="37"/>
</Rule>'
mojodna commented 11 years ago

Can reproduce with cascadenik-2.6.4 and mapnik-0.7.0-2ubuntu1 on Ubuntu 10.04.

mojodna commented 11 years ago

This was fixed in mapnik 0.7.2 (mapnik/mapnik@80aab72438a12479532077af3e4ef3904929f98d).

mojodna commented 11 years ago

FWIW, I've packaged mapnik-0.7.3 for Ubuntu 10.04 (mapnik-0.7.3-1mojodna1~lucid1) and stuck it in a PPA here: https://launchpad.net/~mojodna/+archive/random

springmeyer commented 11 years ago

cool @mojodna. It's not at all obvious (I realize now) but https://launchpad.net/~mapnik/+archive/nightly-0.7/+packages also contains a build of the Mapnik 0.7.x branch which is basically == to the tagged 0.7.3.

springmeyer commented 11 years ago

heh, looking more closely now I see those builds actually failed, so nm!

mojodna commented 11 years ago

I saw those, but assumed that 0.7 support had gone the way of the dodo given the dates and failures, so ripped off the Debian packaging and updated it for 0.7.3.

springmeyer commented 11 years ago

cool, sounds good. yeah, 0.7.x is pretty ancient at this point. Hope you have a good reason for using it ;)

mojodna commented 11 years ago

Not yet, other than potential assumptions for a near-deadline project ;-)