From 6658266e14144854af92770c737a19a75838d9f1 Mon Sep 17 00:00:00 2001
From: Brian Ingles <brianingles@deephaven.io>
Date: Tue, 16 Jan 2024 10:54:34 -0600
Subject: [PATCH] Added jsdoc and removed commented out code

#101
---
 packages/chart/src/ChartUtils.ts | 20 +++++---------------
 1 file changed, 5 insertions(+), 15 deletions(-)

diff --git a/packages/chart/src/ChartUtils.ts b/packages/chart/src/ChartUtils.ts
index d8f6e2a02d..9207334efe 100644
--- a/packages/chart/src/ChartUtils.ts
+++ b/packages/chart/src/ChartUtils.ts
@@ -1872,6 +1872,11 @@ class ChartUtils {
     return axis;
   }
 
+  /**
+   * Creates a plotly layout object based on a given theme.
+   * See https://plotly.com/javascript/reference/layout/
+   * @param theme The theme to use for the layout
+   */
   makeDefaultLayout(theme: ChartTheme): Partial<Layout> {
     const { dh } = this;
 
@@ -1887,21 +1892,6 @@ class ChartUtils {
       lake_color,
       river_color,
       /* eslint-disable camelcase */
-
-      /* Used by `getColorwayFromTheme()` */
-      // colorway,
-
-      /* Used by `makeLayoutAxis()` */
-      // gridcolor,
-      // linecolor,
-      // zerolinecolor,
-
-      // Used by `getPlotlyErrorBars()`
-      // error_band_line_color,
-
-      /* Used by `addStylingToSeriesData()` */
-      // ohlc_increasing,
-      // ohlc_decreasing,
     } = theme;
 
     const layout: Partial<Layout> = {