Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#732 closed defect (fixed)

Advanced placement problems

Reported by: zspitzer Owned by: waltweltonlair
Priority: low Milestone: 2.1
Component: Rendering Service Version: 2.0.2
Severity: major Keywords:
Cc: waltweltonlair External ID:

Description

I have been having some problems with advanced placement

Attached package contains

example 1 : aug O6 ROADS - FWY - WITH ADVANCED PLACEMENT SET LESS THAN ZERO (This shows text scaling in the dynamic state, following the road)

example 2 : aug O6 ROADS - FWY - WITH ADVANCED PLACEMENT SET 1.0 (This shows text still scaling even though it should not scale but remain at the size set)

With Example 2 : If the text is set as 1.0 (do not scale) it should still follow/curve with the road line and only display if it fits within the length of the road, it should not scale to fit. I would prefer text to only display if the length of the text string fits within the road segment length but also follows the curve of the road.

Attachments (3)

ADVANCED PLACEMENT SET LESS THAN ZERO.png (71.7 KB ) - added by zspitzer 16 years ago.
ADVANCED PLACEMENT SET 1.0.png (262.0 KB ) - added by zspitzer 16 years ago.
ADVANCED PLACEMENT test mgp.zip (156.7 KB ) - added by zspitzer 16 years ago.

Download all attachments as: .zip

Change History (11)

by zspitzer, 16 years ago

by zspitzer, 16 years ago

comment:1 by tomfukushima, 16 years ago

Cc: waltweltonlair added

comment:2 by waltweltonlair, 16 years ago

The stylization code ignores the ScaleLimit parameter, and effectively always uses a limit of 0.5. If a path label is too long for the current feature, it scales it by up to a factor of 0.5 to see if it will fit. This happens in RS_FontEngine::LayoutPathText - see the font_scale variable about 30 lines into the method.

Updating the code to honor the ScaleLimit setting would not be too much work (1/2 day at most, is my estimate).

comment:3 by waltweltonlair, 16 years ago

Owner: set to waltweltonlair
Status: newassigned

comment:4 by waltweltonlair, 16 years ago

Milestone: 2.1
Resolution: fixed
Status: assignedclosed

Fixed in trunk with submission https://trac.osgeo.org/mapguide/changeset/3483.

comment:5 by waltweltonlair, 16 years ago

The path labeling code now respects the ScaleLimit parameter. Note that labels which are longer than the length of the feature still get displayed. This problem is not addressed by this ticket / submission.

comment:6 by zspitzer, 16 years ago

Thanks for fixing that, shall I file a new bug about the longer labels?

comment:7 by waltweltonlair, 16 years ago

It's the same problem as in your tiled map where there are straight labels which don't track the roads. If you've already created a ticket for that one then don't create another.

comment:8 by zspitzer, 16 years ago

ok, that's #774

Note: See TracTickets for help on using tickets.