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

Label outline not changing when OUTLINEWIDTH value changed #5107

Merged
merged 1 commit into from Jul 16, 2015

Conversation

Projects
None yet
2 participants
@tbonfort
Member

tbonfort commented Jul 13, 2015

I have a mapfile that has several layers that use SIZEUNITS METERS for styling the layers. Specific to this issue is that the layers are labelled. The text seems to be sized properly in meters, but the outline on the text is not. It seems to always be the same no matter what value I set OUTLINEWIDTH to.

Sample mapfile: https://www.dropbox.com/s/rg9bph3awkmujr2/outlinewidth.map?dl=0

This seems to have started happening after the fix for #4942.

I checked out 3e7ecf7, and the problem did not happen, but checking out d0043f2 makes it happen. It also happens with an up-to-date branch-7-0

@tbonfort

This comment has been minimized.

Show comment
Hide comment
@tbonfort

tbonfort Jul 13, 2015

Member

@cnieman can you confirm all is OK with this patch?

Member

tbonfort commented Jul 13, 2015

@cnieman can you confirm all is OK with this patch?

@cnieman

This comment has been minimized.

Show comment
Hide comment
@cnieman

cnieman Jul 13, 2015

Contributor

I just did a quick test and it looks OK.

It's not exactly the same as my build before the fix for #4942, but I assume that things are a bit wrong in that build for the outline widths.

Contributor

cnieman commented Jul 13, 2015

I just did a quick test and it looks OK.

It's not exactly the same as my build before the fix for #4942, but I assume that things are a bit wrong in that build for the outline widths.

@tbonfort tbonfort merged commit 135997a into mapserver:branch-7-0 Jul 16, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment