#2050 Added Marker Line Style for Line Series#2051
#2050 Added Marker Line Style for Line Series#2051KeithBrio wants to merge 5 commits intooxyplot:developfrom
Conversation
There was a problem hiding this comment.
Thank you for this contribution! There are quite a few issues here, but in principle I think the idea is good. Main issues (only marked up a single instance of each with a specific comment):
- Need to use
this.notation when accessing members dashArrayisn't used in most of the render contexts- Some changes to the csprojs that I think shouldn't be there
- This should have its own example rather than modifying the existing on for
MarkerType(makes it easier for people to find the example and feature, and keeps the example code for the given feature simpler)
I've not run this yet to check the behaviour etc. but will do so when I have more time: obviously it will only work with WPF at the moment.
If you don't have time to fix these issues (which presumably have never affected you) then let us know and I'll try to do all the boring bits that are left over.
|
|
||
| /// <inheritdoc/> | ||
| public void DrawRectangles(IList<OxyRect> rectangles, OxyColor fill, OxyColor stroke, double thickness, EdgeRenderingMode edgeRenderingMode) | ||
| public void DrawRectangles(IList<OxyRect> rectangles, OxyColor fill, OxyColor stroke, double thickness, EdgeRenderingMode edgeRenderingMode, double[] dashArray = null) |
There was a problem hiding this comment.
Unused parameter (issue throughout file)
There was a problem hiding this comment.
@VisualMelon What would your expectation to solve this be here to solve this?
Add a new specific method on IRenderContext and use it appropriately in calling classes?
As I noted initially, I hacked into these changes for my personal project years ago, and not familiar with the architecture or the design philosophy, so any direction is greatly appreciated.
|
All valid comments; I will make some amendments and push tonight. |
|
Looks like |
KeithBrio
force-pushed
the
feature/2050-ability-to-set-marker-line-style-on-line-series
branch
from
92488e2 to
013b5af
Compare
| /// <param name="stroke">The stroke color.</param> | ||
| /// <param name="strokeThickness">The stroke thickness.</param> | ||
| /// <param name="edgeRenderingMode">The edge rendering mode.</param> | ||
| /// <param name="dashArray"></param> |
| /// <param name="thickness">The thickness (in device independent units, 1/96 inch).</param> | ||
| /// <param name="edgeRenderingMode">The edge rendering mode.</param> | ||
| void DrawEllipse(OxyRect extents, OxyColor fill, OxyColor stroke, double thickness, EdgeRenderingMode edgeRenderingMode); | ||
| /// <param name="dashArray"></param> |
| /// <param name="stroke">The stroke color. If set to <c>OxyColors.Undefined</c>, the ellipses will not be stroked.</param> | ||
| /// <param name="thickness">The stroke thickness (in device independent units, 1/96 inch).</param> | ||
| /// <param name="edgeRenderingMode">The edge rendering mode.</param> | ||
| /// <param name="dashArray"></param> |
| /// <param name="thickness">The stroke thickness (in device independent units, 1/96 inch).</param> | ||
| /// <param name="edgeRenderingMode">The edge rendering mode.</param> | ||
| void DrawRectangle(OxyRect rectangle, OxyColor fill, OxyColor stroke, double thickness, EdgeRenderingMode edgeRenderingMode); | ||
| /// <param name="dashArray"></param> |
| /// <param name="stroke">The stroke color. If set to <c>OxyColors.Undefined</c>, the extents will not be stroked.</param> | ||
| /// <param name="thickness">The stroke thickness (in device independent units, 1/96 inch).</param> | ||
| /// <param name="edgeRenderingMode">The edge rendering mode.</param> | ||
| /// <param name="dashArray"></param> |
2 participants
Add the ability to set the Marker Line Style for Line Series charts.
Been sitting on my hard drive for the last couple of years and figured I should push it up.