Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Fup to #119443# one point connector line custom shapes with line width are not well handled when used as 3D shapes | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Impress | Reporter: | Armin Le Grand <Armin.Le.Grand> | ||||||
Component: | open-import | Assignee: | Armin Le Grand <Armin.Le.Grand> | ||||||
Status: | CLOSED FIXED | QA Contact: | |||||||
Severity: | Normal | ||||||||
Priority: | P3 | CC: | lijiany, lijianyuan1983 | ||||||
Version: | 4.0.0-dev | ||||||||
Target Milestone: | 4.0.0 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Attachments: |
|
Description
Armin Le Grand
2012-08-14 10:28:15 UTC
ALG: Taking over. ALG: Problem is that the SdrObjects used to create the visual representation for the custom shape are not converted further when e.g. only lines, but line style is used (e.g. line width like here). It is hard to achieve since it's not enough to do that further conversion, but also properties need to be adapted (e.g. line color to fill color and others). This is tricky since a single SfxItemSet is being used for handling all visualisation SdrObjects. In principle the custom shape visualization would need to be reworked to create primitives. We already can convert primitives to SdrObjects if the custom shape should be 'broken', visualisation is faster, easier and better and attribute handling, too. A custom shape could create a high-level primitive and the custom shape engine stuff could be implemented in decompositions. For now, I'll have to find a tricky way to do this. ALG: Changed to do the needed conversions, works as well as currently possible. Missing: - The first shape has an extra PPT-setting for wireframe, to support that it would need to be imported. - The others show some edge rounding, it also would need to be imported, but principally we support that (see our own 3D objects). It would be easier to support this if the process would be changed to primitive usage. - Error in PPT: The third shape (gray) is not shown transparent despite it is set to 50% transparence. Checking in current solution. "alg" committed SVN revision 1373329 into trunk: #120569# Enhanced 3D visualisation of line custom shapes ALG: Committed as rev 1373329, done. Created attachment 79023 [details]
A continue patch for this issue. Fix wire frame issue.
This patch will fix wire frame issue. Please help review. Thanks.
The wire color is darker than origin. Seems core set the color like that. Any suggestions?
@Armin: An regression is caused by the change in revision 1373329. Import the sample file "Shape.ppt". All Shape 3D effects are lost. Please help check. Thanks. Created attachment 79036 [details]
Shape.ppt
Reopen to fix the regression issue. "alg" committed SVN revision 1376067 into trunk: #120569# Corrected 3D object creations ALG: Found; it is a mix of factors. When the 2D objects get created as base for 3D, more than one object gets created, e.g. for the smiley face. These were handled before the original fix by converting to 3D, but with XLINE_NONE style, thus were never seen. With the fix, these were converted to filled polygons when the line had a width, so got visible. It should only be changed to contour when it's a single object in the 2d representation. Also, this conversion should only be done for non-filled shapes, so first need to check the fill style. Changed this (and some smaller things), also added the nice extension that drawing::ShadeMode_DRAFT is known, thanks to Jianyuan Li! Verified against rev r1377620 on windows 7. The problem is fixed. |