Removed unwanted move around of marker linking first and last data point in

polar coordinates during animation.

Author: simzer

CHANGELOG.md

@@ -8,6 +8,8 @@
 - Fixed unintentional size change of circle markers during polar-cartesian 
   coordinate system change and animation from/to treemap.
 - Fixed line width animation when geometry is changing.
+- Removed unwanted move around of marker linking first and last data point in
+  polar coordinates during animation. 
 
 ## [0.5.1] - 2022-07-14
 

src/chart/rendering/items/areaitem.cpp

@@ -35,7 +35,21 @@ AreaItem::AreaItem(const Diag::Marker &marker,
 		if (prev)
 		{
 			auto prevSpacing = prev->spacing * prev->size / 2;
-			auto prevPos = prev->position - prevSpacing;
+			auto prevPos = prev->position;
+
+			if ((double)options.polar.get() > 0)
+			{
+				if ((double)options.horizontal.get() > 0.5)
+				{
+					if (prevPos.x >= 1) prevPos.x -= 1;
+				}
+				else
+				{
+					if (prevPos.y >= 1) prevPos.y -= 1;
+				}
+			}
+
+			prevPos = prevPos - prevSpacing;
 
 			points[3] = prevPos;
 

src/chart/rendering/items/lineitem.cpp

@@ -40,7 +40,21 @@ LineItem::LineItem(const Diag::Marker &marker,
 		if (prev)
 		{
 			auto prevSpacing = prev->spacing * prev->size / 2;
-			auto prevPos = prev->position - prevSpacing;
+			auto prevPos = prev->position;
+
+			if ((double)options.polar.get() > 0)
+			{
+				if ((double)options.horizontal.get() > 0.5)
+				{
+					if (prevPos.x >= 1) prevPos.x -= 1;
+				}
+				else
+				{
+					if (prevPos.y >= 1) prevPos.y -= 1;
+				}
+			}
+			
+			prevPos = prevPos - prevSpacing;
 
 			lineWidth[0] = std::max(maxWidth * prev->sizeFactor, minWidth);