Issue 102548 - Text adjusted to contour disappears (Draw)
Summary: Text adjusted to contour disappears (Draw)
Status: CLOSED FIXED
Alias: None
Product: Draw
Classification: Application
Component: formatting (show other issues)
Version: OOO310m11
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.1.1
Assignee: wolframgarten
QA Contact: issues@graphics
URL: http://www.youtube.com/watch?v=sdzDC7...
Keywords: oooqa, regression
Depends on:
Blocks:
 
Reported: 2009-06-05 19:47 UTC by fyva
Modified: 2009-07-30 12:24 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
text-shape (10.97 KB, text/plain)
2009-06-06 20:45 UTC, fyva
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description fyva 2009-06-05 19:47:47 UTC
Draw a shape, convert it to curve or contour, add text to it, make this text to
adjust the contour by right clicking on the shape and going to the "Text..."
menu. Text adjusted to contour disappears. It is visible only when you edit it.
Please watch the screencast about this issue: <a
href="http://www.youtube.com/watch?v=sdzDC7oSVDs">YouTube - (?) Text adjusted to
contour</a> .
This is found in DEV300m49 and OOO310m11 on Linux and in OOO310m11 on Windows XP.
Comment 1 Regina Henschel 2009-06-05 20:18:07 UTC
Error as described and shown in video :)

The same file shows the text correct in OOo3.0.1 on WinXP.
Comment 2 fyva 2009-06-06 19:28:36 UTC
This issue is not in OO.o 3.0.1 on Linux and Windows.
Comment 3 Regina Henschel 2009-06-06 19:58:56 UTC
Do not set the target milestone. That will be done by the developer.

Please read http://www.openoffice.org/scdocs/ddIssues_EnterModify.html#priority
to see, that it is not a P2 issue.
Comment 4 fyva 2009-06-06 20:44:46 UTC
Maybe it is not so obvious that it is a P2 issue, because in the example with
the ellipse this functionality is not needed at all. But what about this
text-shape? (see attachment below) Or any other custom text-shape? This issue is
enough important, IMHO, especially because it is not a new feature, but a break.
Comment 5 fyva 2009-06-06 20:45:17 UTC
Created attachment 62828 [details]
text-shape
Comment 6 monstr 2009-07-01 14:55:14 UTC
Exact
Comment 7 Armin Le Grand 2009-07-02 17:39:30 UTC
AW: Checked the polygons who are involved in the primitives; one starting from
the ellipse, the other from the polygon object. These are equal, so there must
be someting else. Looking for DrawOutliner setup differences staring at
SdrTextObj::impDecomposeContourTextPrimitive...
Comment 8 Armin Le Grand 2009-07-02 18:12:09 UTC
AW: Found. Problem is that all the decomposition stuff is layouted for unified
geometry data and an object transformation holding the whole transformation
stack. For historical reasons (the old polygon had only integer, so it colud not
be in unified coordinates, this would have resetted all data to 0 and 1) it's
different for path objects.
Normally the combination of object transformation and unified geometry describes
everything, whereby the scale is in the transformation. For the path object, the
scale is applied to the polygon (geometric data) and is extracted (set to 1.0,
1.0) in the transformation. In the assumed combination, all leads to equal
results (e.g. for getting the real poygon, it gets transformed with the
transformation; it makes no difference where the scale is loacted).
But (of course :-)) there is an exception: The polygon setup for contour text
decomposition in combination with the (old) outliner stuff needs the scale
separated (since it still operates on old integers). Guess where the scale is
taken from...? From the object transformation where it's (1.0, 1.0) for the
polygon case...

There are two solutions:
(a) Change SdrPathPrimitive2D constructor calls (only one) to get the correct
data representations
(b) check if the scale is 1.0/the polygon has some size in
impDecomposeContourTextPrimitive and get the scale from the polygon

While (a) is technically cleaner, (b) is less dangerous. Evaluating...
Comment 9 Armin Le Grand 2009-07-03 11:03:43 UTC
AW: Modifying SdrPathPrimitive2D. There, the basegfx::B2DPolyPolygon member
maUnitPolyPolygon already has the inteded name, so it seems to have the scaling
in the object transformation always was intended. That change will influence the
decompositions of drawinglayer::primitive2d::SdrPathPrimitive2D, so i need to
test some cases. Only visualisations can be influenced, though (due to primitive
usage). Taking a look...
Comment 10 Armin Le Grand 2009-07-03 11:16:49 UTC
AW: Checked line handling thoroughly; especially lines with two points and
bezier polygons with just one point. All works well, also the task itself is solved.
Since for the user this looks like a data loss i will take this on 3.1.1. Adding
to CWS aw074, too...
Comment 11 Armin Le Grand 2009-07-03 11:19:22 UTC
AW: Commited changes, done.
Comment 12 Armin Le Grand 2009-07-06 16:42:24 UTC
AW: Ckecked with wntmsci12.pro build, works as expected.
Comment 13 Armin Le Grand 2009-07-07 09:56:53 UTC
AW->WG: Please review as described.
Comment 14 wolframgarten 2009-07-09 09:30:16 UTC
Verified in CWS.
Comment 15 wolframgarten 2009-07-30 12:24:06 UTC
Tested in m17. Closed.