Bug 400187 - New LineStyle that uses round line caps
Summary: New LineStyle that uses round line caps
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: BIRT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Birt-Chart-inbox@eclipse.org CLA
QA Contact: Xiaoying Gu CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-07 04:49 EST by Lukas M CLA
Modified: 2013-02-18 09:16 EST (History)
1 user (show)

See Also:

Attachments
Image showing a line series with rectangular caps (15.28 KB, image/png)
2013-02-07 04:49 EST, Lukas M CLA 		no flags Details
Patch that adds the new line style to the LineStyle enumerator class (2.46 KB, patch)
2013-02-07 06:08 EST, Lukas M CLA 		no flags Details | Diff
Patch that modifies G2DRendererBase to interpret the new line style (1.28 KB, patch)
2013-02-07 06:09 EST, Lukas M CLA 		no flags Details | Diff
Patch that adds the new line style to attribute.xsd (940 bytes, patch)
2013-02-18 09:16 EST, Lukas M CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lukas M CLA 2013-02-07 04:49:04 EST 
Created attachment 226680 [details]
Image showing a line series with rectangular caps

When using line charts in BIRT each line of the corresponding series is rendered with rectangular caps. This is not a big deal as long as the chart is output as pixel graphic. However if the chart is displayed as vector graphic the intersections between two lines have sharp edges depending on the angle at which they intersect.
In order to remove these artifacts a new LineStyle could be defined which uses round caps instead of rectangular ones.
On the code side the only modifications required are changes in G2DRendererBase, the LineStyle enumeration and corresponding UI classes.
Comment 1 Lukas M CLA 2013-02-07 06:08:08 EST 
Created attachment 226685 [details]
Patch that adds the new line style to the LineStyle enumerator class
Comment 2 Lukas M CLA 2013-02-07 06:09:47 EST 
Created attachment 226686 [details]
Patch that modifies G2DRendererBase to interpret the new line style
Comment 3 Lukas M CLA 2013-02-18 09:16:57 EST 
Created attachment 227201 [details]
Patch that adds the new line style to attribute.xsd

Patch for attribute.xsd that I forgot to attach in the first place.