New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#2050 Added Marker Line Style for Line Series #2051
base: develop
Are you sure you want to change the base?
#2050 Added Marker Line Style for Line Series #2051
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 dashArray
isn'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.
@@ -338,7 +338,7 @@ public void DrawRectangle(OxyRect rectangle, OxyColor fill, OxyColor stroke, dou | |||
} | |||
|
|||
/// <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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused parameter (issue throughout file)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@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 |
92488e2
to
013b5af
Compare
@@ -269,6 +269,7 @@ public static void DrawMultilineText(this IRenderContext rc, ScreenPoint point, | |||
/// <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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✏ missing comment.
@@ -32,7 +32,8 @@ public interface IRenderContext | |||
/// <param name="stroke">The stroke color. If set to <c>OxyColors.Undefined</c>, the extents will not be stroked.</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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✏ missing comment.
@@ -42,10 +43,11 @@ public interface IRenderContext | |||
/// <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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✏ missing comment.
@@ -133,7 +135,8 @@ public interface IRenderContext | |||
/// <param name="stroke">The stroke color. If set to <c>OxyColors.Undefined</c>, the rectangle 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> | |||
void DrawRectangle(OxyRect rectangle, OxyColor fill, OxyColor stroke, double thickness, EdgeRenderingMode edgeRenderingMode); | |||
/// <param name="dashArray"></param> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✏ missing comment.
@@ -143,10 +146,11 @@ public interface IRenderContext | |||
/// <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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✏ missing comment.
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.