Skip to content

Commit

Permalink
text-path: curve > symbol-placement: line, rm symbol-rotation-alignment
Browse files Browse the repository at this point in the history
  • Loading branch information
Lauren Budorick committed Jul 10, 2014
1 parent dea5ce6 commit c94fcdf
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions migrations/v4.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,13 +99,11 @@ function convertLayer(layer) {
rename(render, 'text-min-distance', 'symbol-min-distance');

if (layer.style && layer.style['icon-rotate-anchor']) {
render['symbol-rotation-alignment'] = layer.style['icon-rotate-anchor'];

This comment has been minimized.

Copy link
@mourner

mourner Jul 10, 2014

Contributor

Not sure if this line should go away, @ansis?

This comment has been minimized.

Copy link
@ansis

ansis Jul 10, 2014

Contributor

I think it was right, but I'm not sure. Why remove symbol-rotation-alignment @lbud?

This comment has been minimized.

Copy link
@lbud

lbud Jul 10, 2014

Contributor

oh I have no idea, was confused in what to remove in chat. Re-add?

This comment has been minimized.

Copy link
@mourner

mourner Jul 10, 2014

Contributor

yep

This comment has been minimized.

Copy link
@ansis

ansis Jul 10, 2014

Contributor

yep, I think so

This comment has been minimized.

Copy link
@lbud

lbud Jul 10, 2014

Contributor

thanks, done

delete layer.style['icon-rotate-anchor'];
}

if (render['text-path' === 'curve']) {
render['symbol-rotation-alignment'] = 'map';
render.placement = 'line';
if (render['text-path'] === 'curve') {
render['symbol-placement'] = 'line';
}

delete render['text-path'];
Expand Down

0 comments on commit c94fcdf

Please sign in to comment.