Opened 16 years ago
Closed 15 years ago
#2770 closed defect (fixed)
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 (8)
comment:1 by , 16 years ago
Component: | MapServer C Library → MapServer Documentation |
---|---|
Status: | new → assigned |
comment:2 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | assigned → new |
comment:3 by , 15 years ago
Status: | new → assigned |
---|
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
- updated the mapfile reference's LABEL object for the POSITION parameter
comment:5 by , 15 years ago
Component: | MapServer Documentation → MapServer C Library |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
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
comment:6 by , 15 years ago
Cc: | added |
---|---|
Owner: | changed from | to
Status: | reopened → new |
Jeff: No need to update docs with this, I plan to fix for beta2.
Steve
comment:7 by , 15 years ago
Status: | new → assigned |
---|
comment:8 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed in r8608, closing...
Steve
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