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

Polygon legend problem #4105

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@tbonfort
Member

tbonfort commented Apr 4, 2013

Reporter: zanollim
Date: 2011/12/02 - 10:26
Trac URL: http://trac.osgeo.org/mapserver/ticket/4105
If I define my polygon layer as LINE as follows:

[...]
TYPE LINE
STYLE
  WIDTH 4
  LINECAP round
  OUTLINECOLOR 0 0 0
  PATTERN 0.1 6 END
END
[...]

the legend icon obtained through cgi is correct (see line_legend.png)

If I define the same layer as POLYGON as follows:

[...]
TYPE POLYGON
STYLE
  COLOR 255 255 255
END
STYLE
  WIDTH 4
  LINECAP round
  OUTLINECOLOR 0 0 0
  PATTERN 0.1 6 END
END
[...]

the legend icon is wrong, e.g. points are cut in the middle (see polygon_legend.png)

@mapserver-bot

This comment has been minimized.

Show comment
Hide comment
@mapserver-bot

mapserver-bot commented Apr 5, 2012

line_legend.png

@mapserver-bot

This comment has been minimized.

Show comment
Hide comment

@ghost ghost assigned tbonfort Apr 5, 2012

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort
Member

tbonfort commented Apr 4, 2013

cc/ @zanollim

@sdlime

This comment has been minimized.

Show comment
Hide comment
@sdlime

sdlime Apr 4, 2013

Member

Good catch and fix... --Steve

Member

sdlime commented Apr 4, 2013

Good catch and fix... --Steve

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Apr 8, 2013

Member

rebased and added to master in f87ab84

Member

tbonfort commented Apr 8, 2013

rebased and added to master in f87ab84

@tbonfort tbonfort closed this Apr 8, 2013

tbonfort added a commit to mapserver/msautotest_DEPRECATED that referenced this pull request Apr 9, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment