From 49d0330dff3b020b63169182e607e86f084273d1 Mon Sep 17 00:00:00 2001
From: sophiamersmann <sophia.mersmann1@gmail.com>
Date: Fri, 23 Feb 2024 11:09:06 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9D=20(dod)=20disable=20automatic=20lo?=
 =?UTF-8?q?okup=20for=20indicator=20names?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 adminSiteClient/DimensionCard.tsx             |  7 ---
 .../grapher/src/core/Grapher.tsx              | 61 +++++++++++--------
 2 files changed, 36 insertions(+), 32 deletions(-)

diff --git a/adminSiteClient/DimensionCard.tsx b/adminSiteClient/DimensionCard.tsx
index cbee2070429..43af017c3b1 100644
--- a/adminSiteClient/DimensionCard.tsx
+++ b/adminSiteClient/DimensionCard.tsx
@@ -14,7 +14,6 @@ import {
 } from "@fortawesome/free-solid-svg-icons"
 import { FontAwesomeIcon } from "@fortawesome/react-fontawesome/index.js"
 import { OwidTable } from "@ourworldindata/core-table"
-import { makeDetailIdFromText } from "@ourworldindata/utils"
 
 @observer
 export class DimensionCard extends React.Component<{
@@ -151,12 +150,6 @@ export class DimensionCard extends React.Component<{
                             store={dimension.display}
                             auto={column.displayName}
                             onBlur={this.onChange}
-                            helpText={`Detail: ${makeDetailIdFromText({
-                                text:
-                                    dimension.display.name ??
-                                    column.displayName,
-                                type: "indicator",
-                            })}`}
                             errorMessage={this.props.errorMessages?.displayName}
                         />
                         <BindAutoString
diff --git a/packages/@ourworldindata/grapher/src/core/Grapher.tsx b/packages/@ourworldindata/grapher/src/core/Grapher.tsx
index 074f9e8388d..322b7666e3d 100644
--- a/packages/@ourworldindata/grapher/src/core/Grapher.tsx
+++ b/packages/@ourworldindata/grapher/src/core/Grapher.tsx
@@ -65,7 +65,6 @@ import {
     compact,
     getOriginAttributionFragments,
     sortBy,
-    makeDetailIdFromText,
     extractDetailsFromSyntax,
 } from "@ourworldindata/utils"
 import {
@@ -1261,24 +1260,38 @@ export class Grapher
     @computed get detailsOrderedByReference(): string[] {
         if (typeof window === "undefined") return []
 
-        // if a custom axis label is given, extract details from it;
-        // otherwise, check if a detail exists for the default axis label
-        function extractDetailsFromAxisLabel(
-            configLabel?: string,
-            defaultLabel?: string
-        ): string[] {
-            if (configLabel) {
-                return extractDetailsFromSyntax(configLabel)
-            } else if (defaultLabel) {
-                return [
-                    makeDetailIdFromText({
-                        text: defaultLabel,
-                        type: "indicator",
-                    }),
-                ]
-            }
-            return []
-        }
+        // We are toying with the idea of automatically looking for a detail with a specific prefix
+        // (e.g. grapher_indicator_per-capita-emissions) when an indicator name is used as the default axis label.
+        // This feature is disabled for now, but we might want to enable it in the future.
+        //
+        // // if a custom axis label is given, extract details from it;
+        // // otherwise, check if a detail exists for the default axis label
+        // function extractDetailsFromAxisLabel(
+        //     configLabel?: string,
+        //     defaultLabel?: string
+        // ): string[] {
+        //     if (configLabel) {
+        //         return extractDetailsFromSyntax(configLabel)
+        //     } else if (defaultLabel) {
+        //         return [
+        //             makeDetailIdFromText({
+        //                 text: defaultLabel,
+        //                 type: "indicator",
+        //             }),
+        //         ]
+        //     }
+        //     return []
+        // }
+        //
+        // // extract details from axis labels
+        // const yAxisDetails = extractDetailsFromAxisLabel(
+        //     this.yAxisConfig.label,
+        //     this.defaultYAxisLabel
+        // )
+        // const xAxisDetails = extractDetailsFromAxisLabel(
+        //     this.xAxisConfig.label,
+        //     this.defaultXAxisLabel
+        // )
 
         // extract details from supporting text
         const subtitleDetails = !this.hideSubtitle
@@ -1289,13 +1302,11 @@ export class Grapher
             : []
 
         // extract details from axis labels
-        const yAxisDetails = extractDetailsFromAxisLabel(
-            this.yAxisConfig.label,
-            this.defaultYAxisLabel
+        const yAxisDetails = extractDetailsFromSyntax(
+            this.yAxisConfig.label || ""
         )
-        const xAxisDetails = extractDetailsFromAxisLabel(
-            this.xAxisConfig.label,
-            this.defaultXAxisLabel
+        const xAxisDetails = extractDetailsFromSyntax(
+            this.xAxisConfig.label || ""
         )
 
         // text fragments are ordered by appearance