Opened 10 years ago

Closed 10 years ago

Last modified 10 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 10 years ago.
ADVANCED PLACEMENT SET 1.0.png (262.0 KB) - added by zspitzer 10 years ago.
ADVANCED PLACEMENT test mgp.zip (156.7 KB) - added by zspitzer 10 years ago.

Download all attachments as: .zip

Change History (11)

Changed 10 years ago by zspitzer

Changed 10 years ago by zspitzer

Changed 10 years ago by zspitzer

comment:1 Changed 10 years ago by tomfukushima

Cc: waltweltonlair added

comment:2 Changed 10 years ago by waltweltonlair

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 Changed 10 years ago by waltweltonlair

Owner: set to waltweltonlair
Status: newassigned

comment:4 Changed 10 years ago by waltweltonlair

Milestone: 2.1
Resolution: fixed
Status: assignedclosed

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

comment:5 Changed 10 years ago by waltweltonlair

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 Changed 10 years ago by zspitzer

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

comment:7 Changed 10 years ago by waltweltonlair

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 Changed 10 years ago by zspitzer

ok, that's #774

Note: See TracTickets for help on using tickets.