From c5cd96ae7956319cb5b00c3718208b9602406707 Mon Sep 17 00:00:00 2001
From: Sophia Mersmann <sophia.mersmann1@gmail.com>
Date: Thu, 12 Dec 2024 15:29:41 +0100
Subject: [PATCH] =?UTF-8?q?=E2=9C=A8=20(slope)=20improve=20svg=20output=20?=
 =?UTF-8?q?for=20figma?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../grapher/src/slopeCharts/SlopeChart.tsx    | 126 ++++++++++--------
 1 file changed, 69 insertions(+), 57 deletions(-)

diff --git a/packages/@ourworldindata/grapher/src/slopeCharts/SlopeChart.tsx b/packages/@ourworldindata/grapher/src/slopeCharts/SlopeChart.tsx
index 8907d040e79..cf1e6915dec 100644
--- a/packages/@ourworldindata/grapher/src/slopeCharts/SlopeChart.tsx
+++ b/packages/@ourworldindata/grapher/src/slopeCharts/SlopeChart.tsx
@@ -1143,7 +1143,7 @@ export class SlopeChart
 
     private renderSlopes() {
         return (
-            <>
+            <g id={makeIdForHumanConsumption("slopes")}>
                 {this.renderSeries.map((series) => (
                     <Slope
                         key={series.seriesName}
@@ -1153,7 +1153,28 @@ export class SlopeChart
                         outlineStroke={this.backgroundColor}
                     />
                 ))}
-            </>
+            </g>
+        )
+    }
+
+    private renderInteractiveSlopes(): React.ReactElement {
+        return (
+            <g
+                ref={this.slopeAreaRef}
+                onMouseMove={this.onMouseMove}
+                onTouchMove={this.onMouseMove}
+                onTouchStart={this.onMouseMove}
+                onMouseLeave={this.onMouseLeave}
+            >
+                <rect
+                    x={this.startX}
+                    y={this.bounds.y}
+                    width={this.endX - this.startX}
+                    height={this.bounds.height}
+                    fillOpacity={0}
+                />
+                {this.renderSlopes()}
+            </g>
         )
     }
 
@@ -1178,7 +1199,7 @@ export class SlopeChart
         const { xDomain, startX, endX } = this
 
         return (
-            <>
+            <g id={makeIdForHumanConsumption("horizontal-axis")}>
                 <MarkX
                     label={this.formatColumn.formatTime(xDomain[0])}
                     x={startX}
@@ -1195,32 +1216,6 @@ export class SlopeChart
                     labelPadding={this.xLabelPadding}
                     fontSize={this.yAxis.tickFontSize}
                 />
-            </>
-        )
-    }
-
-    private renderChartArea() {
-        return (
-            <g>
-                {this.renderYAxis()}
-                {this.renderXAxis()}
-                <g
-                    id={makeIdForHumanConsumption("slopes")}
-                    ref={this.slopeAreaRef}
-                    onMouseMove={this.onMouseMove}
-                    onTouchMove={this.onMouseMove}
-                    onTouchStart={this.onMouseMove}
-                    onMouseLeave={this.onMouseLeave}
-                >
-                    <rect
-                        x={this.startX}
-                        y={this.bounds.y}
-                        width={this.endX - this.startX}
-                        height={this.bounds.height}
-                        fillOpacity={0}
-                    />
-                    {this.renderSlopes()}
-                </g>
             </g>
         )
     }
@@ -1286,6 +1281,30 @@ export class SlopeChart
         )
     }
 
+    private renderInteractive(): React.ReactElement {
+        return (
+            <>
+                {this.renderYAxis()}
+                {this.renderXAxis()}
+                {this.renderInteractiveSlopes()}
+                {this.renderLineLegends()}
+                {this.renderNoDataSection()}
+                {this.tooltip}
+            </>
+        )
+    }
+
+    private renderStatic(): React.ReactElement {
+        return (
+            <>
+                {this.renderYAxis()}
+                {this.renderXAxis()}
+                {this.renderSlopes()}
+                {this.renderLineLegends()}
+            </>
+        )
+    }
+
     render() {
         if (this.failMessage)
             return (
@@ -1299,14 +1318,9 @@ export class SlopeChart
                 </>
             )
 
-        return (
-            <g>
-                {this.renderChartArea()}
-                {this.renderLineLegends()}
-                {this.renderNoDataSection()}
-                {this.tooltip}
-            </g>
-        )
+        return this.manager.isStatic
+            ? this.renderStatic()
+            : this.renderInteractive()
     }
 }
 
@@ -1338,10 +1352,7 @@ function Slope({
         hover.background || focus.background ? 0.66 * strokeWidth : strokeWidth
 
     return (
-        <g
-            id={makeIdForHumanConsumption("slope", seriesName)}
-            className="slope"
-        >
+        <g id={makeIdForHumanConsumption(seriesName)} className="slope">
             {showOutline && (
                 <LineWithDots
                     startPoint={startPoint}
@@ -1413,22 +1424,16 @@ function GridLines({ bounds, yAxis }: GridLinesProps) {
             {yAxis.tickLabels.map((tick) => {
                 const y = yAxis.place(tick.value)
                 return (
-                    <g
-                        id={makeIdForHumanConsumption(
-                            "grid-line",
-                            tick.formattedValue
-                        )}
+                    <line
+                        id={makeIdForHumanConsumption(tick.formattedValue)}
                         key={tick.formattedValue}
-                    >
-                        <line
-                            x1={bounds.left + yAxis.width}
-                            y1={y}
-                            x2={bounds.right}
-                            y2={y}
-                            stroke="#ddd"
-                            strokeDasharray="3,2"
-                        />
-                    </g>
+                        x1={bounds.left + yAxis.width}
+                        y1={y}
+                        x2={bounds.right}
+                        y2={y}
+                        stroke="#ddd"
+                        strokeDasharray="3,2"
+                    />
                 )
             })}
         </g>
@@ -1452,7 +1457,14 @@ function MarkX({
 }) {
     return (
         <>
-            <line x1={x} y1={top} x2={x} y2={bottom} stroke="#999" />
+            <line
+                id={makeIdForHumanConsumption(label)}
+                x1={x}
+                y1={top}
+                x2={x}
+                y2={bottom}
+                stroke="#999"
+            />
             <text
                 x={x}
                 y={bottom + labelPadding}