[mapserver-dev] Improve label WRAP and WRAP regression

Lime, Steve D (MNIT) steve.lime at state.mn.us
Thu Aug 9 15:19:48 PDT 2018


Other folks are encouraged to check out Oliver's work in github. Looks good from where I sit.

For the questions:

  * test images should be generated via vagrant
  * since this is fixing a bug I think the PR should be done against 7.2 - maybe even 7.0 assuming that’s where the regression was introduced.

Thoughts?

--Steve

-----Original Message-----
From: mapserver-dev [mailto:mapserver-dev-bounces at lists.osgeo.org] On Behalf Of Oliver Tonnhofer
Sent: Wednesday, August 01, 2018 2:05 AM
To: mapserver-dev at lists.osgeo.org
Subject: Re: [mapserver-dev] Improve label WRAP and WRAP regression

Hi,

just a heads-up. I've created a PR to fix the regression and to allow the user to add the Unicode ZERO WIDTH SPACE character in labels to signify possible wrap positions. These positions are independent of the WRAP character. This way you can wrap "Main-Road" and "Main Road" by adding a ZERO WIDTH SPACE after the space/dash. The ZERO WIDTH SPACE will not show up in the rendered output (whether wrapped or not).

You can insert this character with a PostGIS `replace` call. I'll add an example to the documentation.   
 
The PR is at: https://github.com/mapserver/mapserver/pull/5634


I've included two questions:
• I've added a test. How do I generate the expected image? Most "result images perceptually match, though files differ" on my system.
• The patch is against master but I can rebase it. Where? branch-7-2? branch-7-3?


Regards,
Oliver

-- 
Oliver Tonnhofer  | Omniscale GmbH & Co KG  | https://omniscale.com
OpenStreetMap WMS and tile services         | https://maps.omniscale.com







> On 15. Jun 2018, at 13:05, Oliver Tonnhofer <olt at omniscale.de> wrote:
> 
> Hi all,
> 
> we want to improve label wrapping in MapServer so that road labels can be wrapped at space and dashes.
> 
> I created an issue with a request for comments:
> https://github.com/mapserver/mapserver/issues/5613
> 
> I also found a regression that is related to this:
> https://github.com/mapserver/mapserver/issues/5612
> 
> 
> Regards,
> Oliver
> 
> -- 
> Oliver Tonnhofer  | Omniscale GmbH & Co KG  | https://omniscale.com
> OpenStreetMap WMS and tile services         | https://maps.omniscale.com
> 
> 
> 
> 
> 
> 
> 

_______________________________________________
mapserver-dev mailing list
mapserver-dev at lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/mapserver-dev


More information about the mapserver-dev mailing list