Issue 98289 - charts are not sharp / in metafiles also horizontal and vertical lines get antialized
Summary: charts are not sharp / in metafiles also horizontal and vertical lines get an...
Status: CLOSED FIXED
Alias: None
Product: Draw
Classification: Application
Component: viewing (show other issues)
Version: DEV300m38
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.1
Assignee: wolframgarten
QA Contact: issues@graphics
URL:
Keywords: regression
: 99481 (view as issue list)
Depends on:
Blocks:
 
Reported: 2009-01-20 16:36 UTC by IngridvdM
Modified: 2009-02-26 14:10 UTC (History)
1 user (show)

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


Attachments
example showing the problem (17.71 KB, application/vnd.oasis.opendocument.spreadsheet)
2009-01-20 16:37 UTC, IngridvdM
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description IngridvdM 2009-01-20 16:36:18 UTC
Load the attached document or create a simple chart and leave the edit mode.
Since dev300m39 the created meta file looks diffuse.
It seems as if even horizontal and vertical lines are antialized and/or
stretched somehow.
When deselecting the anti-aliasing option the problem is gone.
When activating the chart within the edit mode the problem is also gone. In edit
mode the horizontal and vertical lines are sharp and not antialized whereas the
diagonal line is nicely antialized.
The same visible result should be generated by the metafile.
Comment 1 IngridvdM 2009-01-20 16:37:41 UTC
Created attachment 59533 [details]
example showing the problem
Comment 2 Armin Le Grand 2009-01-21 11:19:44 UTC
AW: The example shows that somehow metafiles and charts (activated) are handled
different. Checked this and it seems as if VCL still does not use the AA and
direct system shortcuts everywhere, so for the activated chart the polygon(s)
get converted to vcl:polygon and back once in a cascade of DrawPolyLine calls.
This lets the points snap to pixels which leads to not-AAed horizontal and
vertical lines.
This needs to be fixed first to get a homogen AAing in all situations. 2nd
thought is to add a 'pixel snap' for horizontal and vertical lines to improve
visualisation for charts, but this may be done for everything (need to evaluate).
First checking VCL, will need HDUs opinion on this.
Comment 3 Armin Le Grand 2009-01-21 16:32:04 UTC
AW: Okay, VCL changes are okay (asked HDU), the #ifded UNX is a relict from the
phase of adding the needed features.
I will opt for adding support for snapping Horizontal/Vertical lines to discrete
pixels at the renderers. It's not an option which should be part of VCL; we plan
anyways to go more away from it with the DrawingLayer. I will also add a feature
to the Drawinglayer configuration to be able to switch this off on demand. The
RenderPolygonHairlinePrimitive2D will when the feature is turned on take needed
actions.
Comment 4 Armin Le Grand 2009-01-22 11:40:08 UTC
AW: Added option to SvtOptionsDrawingLayer, added tooling to basegfx, added
usage of it in vcl renderer. Works as expected for DrawingLayer, BUT not for
MetaFiles. This is because these are solely played from VCL. Argh.
Comment 5 Armin Le Grand 2009-01-22 12:27:17 UTC
AW: Problem with MetaFiles not solvable outside VCL as long as these are not
decomposed (this is the direction for the near future). Added a
ANTIALIASING_PIXELSNAPHAIRLINE flag for OutputDevice::SetAntialiasing, using it
in PRIMITIVE2D_ID_METAFILEPRIMITIVE2D case in
VclPixelProcessor2D::processBasePrimitive2D to explicitely set it for MetaFile
rendering. Usage is in two OutputDevice::DrawPolyLine methods in VCL.
Works well, checking some more...
Comment 6 Armin Le Grand 2009-01-22 13:21:28 UTC
AW: Works well, all visualisations profit from that feature, so default is on
for all geometry. Checking in. Changes in basegfx, drawinglayer, vcl, officecfg
und svtools. Checking currently not possible, need to wait.
Comment 7 Armin Le Grand 2009-01-22 13:22:32 UTC
AW: BTW: This is not broken, but a new enhancement mainly for charts. AA works
as defined. Changing entries accordingly.
Comment 8 Armin Le Grand 2009-01-23 15:19:36 UTC
AW: Checkin done, fininshed.
Comment 9 Armin Le Grand 2009-01-30 11:04:20 UTC
AW->WG: Please verify. Only has effects when AA is on and forces horizontal and
vertical lines to snap to pixels, so that they are not blurred by AAing them.
Comment 10 wolframgarten 2009-01-30 12:52:39 UTC
Verified in CWS.
Comment 11 IngridvdM 2009-02-23 08:48:05 UTC
*** Issue 99481 has been marked as a duplicate of this issue. ***
Comment 12 wolframgarten 2009-02-26 10:02:04 UTC
Tested in OOO310_m3. Closed.
Comment 13 IngridvdM 2009-02-26 14:10:00 UTC
Not all cases are fixed. Create follow up issues: issue 99662 and issue 99665.