Ticket #2770 (closed defect: fixed)

Opened 5 years ago

Last modified 4 years ago

POSITION AUTO for Polygons should include CC (and start with it)

Reported by: sdlime Owned by: sdlime
Priority: normal Milestone: 5.4 release
Component: MapServer C Library Version: unspecified
Severity: normal Keywords:
Cc: jmckenna

Description

Actually we need to decouple that code (msDrawLabelCache) in mapdraw.c from the positions enum. It should be easy, just create a positions int array with a length equal to the number of positions and fill it explicitly with the necessary positions, and save the number loaded. Then we loop through that instead of the start, stop crap currently used.

Steve

Change History

Changed 5 years ago by sdlime

  • status changed from new to assigned
  • component changed from MapServer C Library to MapServer Documentation

Fixed in r7920. Polygons us CC, UC, LC, CL and CR as possible positions. Point/annotation and line layers remain "as is". Assigning to the documentation component.

Steve

Changed 4 years ago by jmckenna

  • owner changed from sdlime to jmckenna
  • status changed from assigned to new

Changed 4 years ago by jmckenna

  • status changed from new to assigned

Changed 4 years ago by jmckenna

  • status changed from assigned to closed
  • resolution set to fixed

- updated the mapfile reference's LABEL object for the POSITION parameter

Changed 4 years ago by sdlime

  • status changed from closed to reopened
  • resolution fixed deleted
  • component changed from MapServer Documentation to MapServer C Library

There's still an issue if the polygon geometry is part of an annotation layer. The geometry type is not available to the label cache code so we don't know if how to set the positions correctly with annotation layers. Currently it treats 'em like points with the 8 positions around the point. We'll need to track the geometry type as part of msAddLabel() to fix this.

Steve

Changed 4 years ago by sdlime

  • cc jmckenna added
  • owner changed from jmckenna to sdlime
  • status changed from reopened to new

Jeff: No need to update docs with this, I plan to fix for beta2.

Steve

Changed 4 years ago by sdlime

  • status changed from new to assigned

Changed 4 years ago by sdlime

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in r8608, closing...

Steve

Note: See TracTickets for help on using tickets.