Merge remote-tracking branch 'origin/axis_refactor_v2' into axis_refactor_v3

Author: schaumb

.github/workflows/docker-vizzu-dev-desktop.yml

@@ -24,6 +24,6 @@ jobs:
             - name: Build and Publish
               run: |
                   IMAGE="vizzu-dev-desktop"
-                  IMAGE_NAME="vizzu/$IMAGE:0.14"
+                  IMAGE_NAME="vizzu/$IMAGE:0.15"
                   docker build -t $IMAGE_NAME -f tools/ci/docker/$IMAGE .
                   docker push $IMAGE_NAME

.github/workflows/docker-vizzu-dev-wasm.yml

@@ -24,6 +24,6 @@ jobs:
             - name: Build and Publish
               run: |
                   IMAGE="vizzu-dev-wasm"
-                  IMAGE_NAME="vizzu/$IMAGE:0.14"
+                  IMAGE_NAME="vizzu/$IMAGE:0.15"
                   docker build -t $IMAGE_NAME -f tools/ci/docker/$IMAGE .
                   docker push $IMAGE_NAME

CHANGELOG.md

@@ -4,16 +4,25 @@
 
 ### Fixed
 
+- Do not interpolate hiding/showing legend
+
+## [0.15.0] - 2024-10-28
+
+### Fixed
+
 - Markers are the same even if new record added.
+- Flying out marker label fixed.
 - Axis line hide/show at same time with axis labels/ticks/title.
 - Do not draw invisible axis line.
-- Do not interpolate hiding/showing legend
 
 ### Changed
 
 - Removed 'min' align property from the API which equivalent with the 'none'.
 - Changed MarkerId to be a string instead of a number.
 
+### Added
+
+- Add marker top and center position to draw event.
 
 ## [0.14.0] - 2024-10-03
 

CONTRIBUTING.md

@@ -70,7 +70,7 @@ docker run -i -t -v .:/workspace vizzu/vizzu-dev-desktop bash
 or you can use a specific version of the prebuilt image:
 
 ```sh
-docker run -i -t -v .:/workspace vizzu/vizzu-dev-desktop:0.14 bash
+docker run -i -t -v .:/workspace vizzu/vizzu-dev-desktop:0.15 bash
 ```
 
 Run the following commands to build and run the `WASM` version's development
@@ -84,7 +84,7 @@ docker run -i -t -v .:/workspace vizzu/vizzu-dev-wasm bash
 or you can use a specific version of the prebuilt image:
 
 ```sh
-docker run -i -t -v .:/workspace vizzu/vizzu-dev-wasm:0.14 bash
+docker run -i -t -v .:/workspace vizzu/vizzu-dev-wasm:0.15 bash
 ```
 
 ### Building the project

src/apps/weblib/ts-api/events.ts

@@ -137,11 +137,19 @@ export interface Logo extends Element {
 export interface Area extends Element {
 	tagName: 'plot-area'
 }
+
+/** Marker element position structure helper for tooltip */
+export interface MarkerPosition {
+	top: Point
+	center: Point
+}
+
 /** Plot marker element of the chart representing a data point. */
 export interface Marker extends Element {
 	tagName: 'plot-marker'
 	categories: Data.Record
 	values: Data.Record
+	position: MarkerPosition
 	/** Unique index of the marker. */
 	index: string
 }

src/base/geom/rect.cpp

@@ -3,6 +3,7 @@
 #include <algorithm>
 #include <array>
 #include <cmath>
+#include <compare>
 
 #include "base/math/floating.h"
 
@@ -92,10 +93,19 @@ Rect Rect::intersection(const Rect &rect) const
 
 bool Rect::intersects(const Rect &r) const
 {
-	using Math::Floating::less;
-	auto isOutside =
-	    less(right(), r.left()) || less(r.right(), left())
-	    || less(top(), r.bottom()) || less(r.top(), bottom());
+	using Math::Floating::is_zero;
+	using std::strong_order;
+	auto first = strong_order(right(), r.left());
+	auto second = strong_order(r.right(), left());
+	auto third = strong_order(top(), r.bottom());
+	auto fourth = strong_order(r.top(), bottom());
+
+	auto isOutside = is_lt(first) || is_lt(second) || is_lt(third)
+	              || is_lt(fourth)
+	              || ((is_eq(first) || is_eq(second))
+	                  && !is_zero(width()) && !is_zero(r.width()))
+	              || ((is_eq(third) || is_eq(fourth))
+	                  && !is_zero(height()) && !is_zero(r.height()));
 	return !isOutside;
 }
 

src/chart/main/events.h

@@ -247,16 +247,23 @@ class Events
 		struct Marker : Element
 		{
 			const Gen::Marker ▮
+			struct DataPosition
+			{
+				Geom::Point top;
+				Geom::Point center;
+			} position;
 
-			explicit Marker(const Gen::Marker &marker) :
+			explicit Marker(const Gen::Marker &marker,
+			    const DataPosition &position) :
 			    Element("plot-marker"),
-			    marker(marker)
+			    marker(marker),
+			    position(position)
 			{}
 
 			void appendToJSON(Conv::JSONObj &&jsonObj) const override
 			{
-				Element::appendToJSON(
-				    marker.appendToJSON(std::move(jsonObj)));
+				Element::appendToJSON(marker.appendToJSON(
+				    std::move(jsonObj))("position", position));
 			}
 		};
 
@@ -268,8 +275,10 @@ class Events
 		{
 			bool horizontal;
 
-			MarkerGuide(const Gen::Marker &marker, bool horizontal) :
-			    MarkerChild("guide", marker),
+			MarkerGuide(const Gen::Marker &marker,
+			    const Marker::DataPosition &position,
+			    bool horizontal) :
+			    MarkerChild("guide", marker, position),
 			    horizontal(horizontal)
 			{}
 
@@ -313,15 +322,19 @@ class Events
 			return std::make_unique<Legend>(properties);
 		}
 
-		static auto marker(const Gen::Marker &marker)
+		static auto marker(const Gen::Marker &marker,
+		    const Marker::DataPosition &position)
 		{
-			return std::make_unique<Marker>(marker);
+			return std::make_unique<Marker>(marker, position);
 		}
 
 		static auto markerGuide(const Gen::Marker &marker,
+		    const Marker::DataPosition &position,
 		    bool horizontal)
 		{
-			return std::make_unique<MarkerGuide>(marker, horizontal);
+			return std::make_unique<MarkerGuide>(marker,
+			    position,
+			    horizontal);
 		}
 
 		static auto root()
@@ -362,11 +375,13 @@ class Events
 		}
 
 		static auto markerLabel(const std::string &label,
-		    const Gen::Marker &marker)
+		    const Gen::Marker &marker,
+		    const Marker::DataPosition &position)
 		{
 			return std::make_unique<Text<MarkerChild>>(label,
 			    "label",
-			    marker);
+			    marker,
+			    position);
 		}
 
 		static auto dimLegendLabel(

src/chart/main/version.cpp

@@ -2,6 +2,6 @@
 
 #include "base/app/version.h"
 
-const App::Version Vizzu::Main::version(0, 14, 0);
+const App::Version Vizzu::Main::version(0, 15, 0);
 
 const char *const Vizzu::Main::siteUrl = "https://vizzu.io/";

src/chart/rendering/markerrenderer.cpp

@@ -66,7 +66,9 @@ void MarkerRenderer::drawLines(Gfx::ICanvas &canvas,
 			const Geom::Line line(axisPoint, blended.center);
 
 			auto guideElement =
-			    Events::Targets::markerGuide(blended.marker, false);
+			    Events::Targets::markerGuide(blended.marker,
+			        blended.dataPosition,
+			        false);
 
 			if (rootEvents.draw.plot.marker.guide->invoke(
 			        Events::OnLineDrawEvent(*guideElement,
@@ -87,7 +89,9 @@ void MarkerRenderer::drawLines(Gfx::ICanvas &canvas,
 			const Geom::Line line(center, axisPoint);
 
 			auto guideElement =
-			    Events::Targets::markerGuide(blended.marker, true);
+			    Events::Targets::markerGuide(blended.marker,
+			        blended.dataPosition,
+			        true);
 
 			if (rootEvents.draw.plot.marker.guide->invoke(
 			        Events::OnLineDrawEvent(*guideElement,
@@ -268,7 +272,8 @@ void MarkerRenderer::draw(Gfx::ICanvas &canvas,
 	canvas.setLineWidth(*rootStyle.plot.marker.borderWidth);
 
 	auto markerElement =
-	    Events::Targets::marker(abstractMarker.marker);
+	    Events::Targets::marker(abstractMarker.marker,
+	        abstractMarker.dataPosition);
 
 	auto colorAlpha =
 	    Math::FuzzyBool::And<double>(abstractMarker.enabled, factor);
@@ -369,7 +374,9 @@ void MarkerRenderer::drawLabel(Gfx::ICanvas &canvas,
 	    Gfx::ColorTransform::OverrideColor(
 	        (*labelStyle.filter)(color)*colorAlpha),
 	    *rootEvents.draw.plot.marker.label,
-	    Events::Targets::markerLabel(text, marker));
+	    Events::Targets::markerLabel(text,
+	        marker,
+	        abstractMarker.dataPosition));
 }
 
 std::string MarkerRenderer::getLabelText(

src/chart/rendering/markers/abstractmarker.cpp

@@ -27,6 +27,7 @@ AbstractMarker AbstractMarker::create(const DrawingContext &ctx,
 	switch (shapeType) {
 	case Gen::ShapeType::rectangle:
 		return RectangleMarker(marker,
+		    ctx.coordSys,
 		    ctx.getOptions(),
 		    ctx.rootStyle);
 	case Gen::ShapeType::circle:
@@ -111,6 +112,17 @@ AbstractMarker AbstractMarker::createInterpolated(
 	return aMarker;
 }
 
+void AbstractMarker::setDataPosition(const CoordinateSystem &coordSys)
+{
+	dataPosition = {
+	    this->getLabelPos(Styles::MarkerLabel::Position::top,
+	            coordSys)
+	        .end,
+	    this->getLabelPos(Styles::MarkerLabel::Position::center,
+	            coordSys)
+	        .begin};
+}
+
 Geom::Rect AbstractMarker::getBoundary() const
 {
 	return Geom::Rect::Boundary(points);