Issue 79917 - Regression: messed-up curves in slideshow view
Summary: Regression: messed-up curves in slideshow view
Status: CLOSED FIXED
Alias: None
Product: Impress
Classification: Application
Component: code (show other issues)
Version: recent-trunk
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.3
Assignee: christian.guenther
QA Contact: issues@graphics
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2007-07-23 17:00 UTC by thb
Modified: 2007-08-13 16:45 UTC (History)
2 users (show)

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


Attachments
bugdoc (11.58 KB, application/vnd.oasis.opendocument.presentation)
2007-07-23 17:00 UTC, thb
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description thb 2007-07-23 17:00:06 UTC
Seems aw051 indeed broke something: load to-be attached bugdoc in m222, start
slideshow to see what's wrong.
Comment 1 thb 2007-07-23 17:00:42 UTC
Created attachment 47002 [details]
bugdoc
Comment 2 thb 2007-07-23 17:01:19 UTC
Set target & keyword.
Comment 3 Armin Le Grand 2007-07-26 11:31:02 UTC
AW: Resynching aw052 to get a working base on m222...
Comment 4 Armin Le Grand 2007-07-26 16:43:44 UTC
AW: Also happens on bitmap-filled curves without outlines (Create ellipse,
convert to curve, no line, bitmap fill). Discussed with THB, i need canvas,
cppcancas and basegfx. Somewhere a polygon transformation is  still wrong...
Comment 5 Armin Le Grand 2007-07-30 09:46:59 UTC
AW: Two reasons found:
- In vcl/source/helper/canvastools.cxx conversions between tools
Polygon/PolyPolygon and RealBezierSegment2D were used which were not
adapted/synchronized to/with the new B2DPolygon formats. These were replaced
with calls to the appropriate methods in the basegfx tooling. This is safer and
better to maintain. Also guarantees correct transport of the last point(s) and
open/closed stuff.
- in basegfx/source/tools/canvastools.cxx, polygonFromBezier2DSequence had two
errors, one took the last instead of the first segment for curve start, the
other used two times the x value for the 2nd control point (whis was the main
reason for the curves looking different).
AW: Both fixed, installsets built.
Comment 6 Armin Le Grand 2007-07-30 10:53:16 UTC
AW->CGU: Please verify. Use the attached example or simply a bitmap-filled ellipse.
Comment 7 christian.guenther 2007-08-02 10:13:36 UTC
CGU: Verified in cws aw052
Comment 8 christian.guenther 2007-08-13 16:45:17 UTC
CGU: integrated in src680m225