Baseline fixes

pfizer-opensource · Oct 10, 2024 · 202b6d9 · 202b6d9
1 parent c5986f6
commit 202b6d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/graphs/moving-range/MovingRangeRenderer.js b/src/graphs/moving-range/MovingRangeRenderer.js
@@ -18,8 +18,8 @@ class MovingRangeRenderer extends ScatterplotRenderer {
   renderGraph(graphElementSelector) {
     this.drawSvg(graphElementSelector);
     this.drawAxes();
-    this.computeGraphLimits();
     this.drawArea();
+    this.computeGraphLimits();
     this.drawGraphLimits(this.y);
     this.setupMouseLeaveHandler();
   }

diff --git a/src/graphs/scatterplot/ScatterplotRenderer.js b/src/graphs/scatterplot/ScatterplotRenderer.js
@@ -603,13 +603,12 @@ class ScatterplotRenderer extends UIControlsRenderer {
         .attr('id', 'text-' + id)
         .style('font-size', '12px');
     }
-
-    lineEl.attr('y1', yScale(yValue)).attr('y2', yScale(yValue)).attr('stroke', color);
+    lineEl.attr('y1', this.applyYScale(yScale,yValue)).attr('y2', this.applyYScale(yScale,yValue)).attr('stroke', color);
     if (text) {
       textEl
         .text(text)
         .attr('fill', color)
-        .attr('y', yScale(yValue) - 4);
+        .attr('y', this.applyYScale(yScale,yValue)-4);
       // Measure text width
       const textWidth = this.#getTextWidth(text, '12px');
       const adjustedX = this.width - textWidth;