From d99afae38508fff359e5228f4501665f2372decc Mon Sep 17 00:00:00 2001
From: Stephen Davies <steve9164@gmail.com>
Date: Mon, 18 Dec 2023 15:40:38 +1100
Subject: [PATCH] Fix missing instructions and chartable item description in
 explorer window

---
 CHANGES.md                             |  1 +
 lib/ReactViews/Preview/DataPreview.jsx | 42 ++++++++++++++------------
 2 files changed, 24 insertions(+), 19 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index e7d0fe15cb2..f3ec5ff3aa8 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -8,6 +8,7 @@
 - Improve tsconfig files
 - Update `thredds-catalog-crawler` to `0.0.6`
 - `WebMapServiceCatalogItem` will drop problematic query parameters from `url` when calling `GetCapabilities` (eg `"styles","srs","crs","format"`)
+- Fixed regression causing explorer window not to display instructions when first opened.
 - [The next improvement]
 
 #### 8.4.1 - 2023-12-08
diff --git a/lib/ReactViews/Preview/DataPreview.jsx b/lib/ReactViews/Preview/DataPreview.jsx
index 027bebdcd5c..34e6020efee 100644
--- a/lib/ReactViews/Preview/DataPreview.jsx
+++ b/lib/ReactViews/Preview/DataPreview.jsx
@@ -68,20 +68,22 @@ class DataPreview extends React.Component {
         </div>
       );
     } else if (chartData) {
-      <div className={Styles.previewInner}>
-        <h3 className={Styles.h3}>{previewed.name}</h3>
-        <p>{t("preview.doesNotContainGeospatialData")}</p>
-        <div className={Styles.previewChart}>
-          {/* TODO: Show a preview chart
+      return (
+        <div className={Styles.previewInner}>
+          <h3 className={Styles.h3}>{previewed.name}</h3>
+          <p>{t("preview.doesNotContainGeospatialData")}</p>
+          <div className={Styles.previewChart}>
+            {/* TODO: Show a preview chart
                 <Chart
                    data={chartData}
                    axisLabel={{ x: previewed.xAxis.units, y: undefined }}
                    height={250 - 34}
                    />
             */}
+          </div>
+          <Description item={previewed} />
         </div>
-        <Description item={previewed} />
-      </div>;
+      );
     } else if (previewed && CatalogFunctionMixin.isMixedInto(previewed)) {
       return (
         <InvokeFunction
@@ -101,18 +103,20 @@ class DataPreview extends React.Component {
         </div>
       );
     } else {
-      <div className={Styles.placeholder}>
-        <Trans i18nKey="preview.selectToPreview">
-          <p>Select a dataset to see a preview</p>
-          <p>- OR -</p>
-          <button
-            className={Styles.btnBackToMap}
-            onClick={() => this.backToMap()}
-          >
-            Go to the map
-          </button>
-        </Trans>
-      </div>;
+      return (
+        <div className={Styles.placeholder}>
+          <Trans i18nKey="preview.selectToPreview">
+            <p>Select a dataset to see a preview</p>
+            <p>- OR -</p>
+            <button
+              className={Styles.btnBackToMap}
+              onClick={() => this.backToMap()}
+            >
+              Go to the map
+            </button>
+          </Trans>
+        </div>
+      );
     }
   }