From 61d7a3d3f4f2a32e196f2d067e592aa5b2e20fd1 Mon Sep 17 00:00:00 2001
From: Brian Ward <bward@flatironinstitute.org>
Date: Mon, 29 Jul 2024 18:09:39 +0000
Subject: [PATCH] Move toolbar to own file, tighter padding when collapsed

---
 gui/src/app/FileEditor/StanFileEditor.tsx |   3 +-
 gui/src/app/FileEditor/TextEditor.tsx     | 131 +---------------------
 gui/src/app/FileEditor/ToolBar.tsx        | 125 +++++++++++++++++++++
 gui/src/app/Scripting/ScriptEditor.tsx    |   3 +-
 gui/src/localStyles.css                   |   3 +
 5 files changed, 134 insertions(+), 131 deletions(-)
 create mode 100644 gui/src/app/FileEditor/ToolBar.tsx

diff --git a/gui/src/app/FileEditor/StanFileEditor.tsx b/gui/src/app/FileEditor/StanFileEditor.tsx
index 6f06b909..47e1a081 100644
--- a/gui/src/app/FileEditor/StanFileEditor.tsx
+++ b/gui/src/app/FileEditor/StanFileEditor.tsx
@@ -1,7 +1,8 @@
 import { AutoFixHigh, Cancel, Settings } from "@mui/icons-material";
 import { SplitDirection, Splitter } from "@SpComponents/Splitter";
 import StanCompileResultWindow from "@SpComponents/StanCompileResultWindow";
-import TextEditor, { ToolbarItem } from "@SpComponents/TextEditor";
+import TextEditor from "@SpComponents/TextEditor";
+import { ToolbarItem } from "@SpComponents/ToolBar";
 import compileStanProgram from "@SpStanc/compileStanProgram";
 import { stancErrorsToCodeMarkers } from "@SpStanc/Linting";
 import useStanc from "@SpStanc/useStanc";
diff --git a/gui/src/app/FileEditor/TextEditor.tsx b/gui/src/app/FileEditor/TextEditor.tsx
index c2843768..609b7c08 100644
--- a/gui/src/app/FileEditor/TextEditor.tsx
+++ b/gui/src/app/FileEditor/TextEditor.tsx
@@ -1,9 +1,8 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { Editor, loader, useMonaco } from "@monaco-editor/react";
-import { Save } from "@mui/icons-material";
-import Button from "@mui/material/Button";
-import Link from "@mui/material/Link";
+
 import monacoAddStanLang from "@SpComponents/stanLang";
+import { ToolBar, ToolbarItem } from "@SpComponents/ToolBar";
 import { CodeMarker } from "@SpStanc/Linting";
 import { editor, MarkerSeverity } from "monaco-editor";
 import {
@@ -30,21 +29,6 @@ type Props = {
   contentOnEmpty?: string | HTMLSpanElement;
 };
 
-export type ToolbarItem =
-  | {
-      type: "button";
-      tooltip?: string;
-      label?: string;
-      icon?: any;
-      onClick: () => void;
-      color?: string;
-    }
-  | {
-      type: "text";
-      label: string;
-      color?: string;
-    };
-
 const TextEditor: FunctionComponent<Props> = ({
   text,
   onSaveText,
@@ -173,117 +157,6 @@ const toMonacoMarkerSeverity = (
   }
 };
 
-type ToolbarProps = {
-  items: ToolbarItem[];
-  label: string;
-  onSaveText: () => void;
-  edited: boolean;
-  readOnly: boolean;
-};
-
-const ToolBar: FunctionComponent<ToolbarProps> = ({
-  items,
-  label,
-  onSaveText,
-  edited,
-  readOnly,
-}) => {
-  const toolBarItems = useMemo(() => {
-    const editorItems: ToolbarItem[] = [];
-
-    if (!readOnly && edited) {
-      editorItems.push({
-        type: "button",
-        icon: <Save />,
-        onClick: onSaveText,
-        tooltip: "Save file",
-        label: "Save",
-      });
-    }
-
-    if (edited) {
-      editorItems.push({
-        type: "text",
-        label: "Edited",
-        color: "red",
-      });
-    }
-
-    if (readOnly) {
-      editorItems.push({
-        type: "text",
-        label: "Read Only",
-        color: "gray",
-      });
-    }
-
-    return editorItems.concat(items);
-  }, [edited, items, onSaveText, readOnly]);
-
-  return (
-    <div className="NotSelectable">
-      <div className="EditorMenuBar">
-        <span className="EditorTitle">{label}</span>
-        {toolBarItems &&
-          toolBarItems.map((item, i) => (
-            <ToolbarItemComponent key={i} item={item} />
-          ))}
-      </div>
-    </div>
-  );
-};
-
-const ToolbarItemComponent: FunctionComponent<{ item: ToolbarItem }> = ({
-  item,
-}) => {
-  if (item.type === "button") {
-    const { onClick, color, label, tooltip, icon } = item;
-    if (icon) {
-      return (
-        <span className="EditorToolbarItem" style={{ color }}>
-          <Button
-            startIcon={icon}
-            onClick={onClick}
-            disabled={!onClick}
-            color="inherit"
-            size="small"
-            title={tooltip}
-          >
-            {label && <span className="ToolbarButtonText">{label}</span>}
-          </Button>
-        </span>
-      );
-    } else {
-      return (
-        <span className="EditorToolbarItem">
-          <Link
-            onClick={onClick}
-            color={color || "gray"}
-            component="button"
-            underline="none"
-            title={tooltip}
-          >
-            {label}
-          </Link>
-          &nbsp;&nbsp;&nbsp;
-        </span>
-      );
-    }
-  } else if (item.type === "text") {
-    return (
-      <span
-        className="EditorToolbarItem"
-        style={{ color: item.color || "gray" }}
-        title={item.label}
-      >
-        {item.label}&nbsp;&nbsp;&nbsp;
-      </span>
-    );
-  } else {
-    return <span>unknown toolbar item type</span>;
-  }
-};
-
 const createHintTextContentWidget = (content: string | HTMLSpanElement) => {
   return {
     getDomNode: () => {
diff --git a/gui/src/app/FileEditor/ToolBar.tsx b/gui/src/app/FileEditor/ToolBar.tsx
new file mode 100644
index 00000000..51ced36b
--- /dev/null
+++ b/gui/src/app/FileEditor/ToolBar.tsx
@@ -0,0 +1,125 @@
+import { FunctionComponent, useMemo } from "react";
+import { Save } from "@mui/icons-material";
+import Link from "@mui/material/Link";
+import IconButton from "@mui/material/IconButton";
+
+export type ToolbarItem =
+  | {
+      type: "button";
+      tooltip?: string;
+      label?: string;
+      icon?: any;
+      onClick: () => void;
+      color?: string;
+    }
+  | {
+      type: "text";
+      label: string;
+      color?: string;
+    };
+
+type ToolbarProps = {
+  items: ToolbarItem[];
+  label: string;
+  onSaveText: () => void;
+  edited: boolean;
+  readOnly: boolean;
+};
+
+export const ToolBar: FunctionComponent<ToolbarProps> = ({
+  items,
+  label,
+  onSaveText,
+  edited,
+  readOnly,
+}) => {
+  const toolBarItems = useMemo(() => {
+    const editorItems: ToolbarItem[] = [];
+
+    if (readOnly) {
+      editorItems.push({
+        type: "text",
+        label: "Read Only",
+        color: "gray",
+      });
+    } else if (edited) {
+      editorItems.push({
+        type: "button",
+        icon: <Save />,
+        onClick: onSaveText,
+        tooltip: "Save file",
+        label: "Save",
+      });
+      editorItems.push({
+        type: "text",
+        label: "Edited",
+        color: "red",
+      });
+    }
+
+    return editorItems.concat(items);
+  }, [edited, items, onSaveText, readOnly]);
+
+  return (
+    <div className="NotSelectable">
+      <div className="EditorMenuBar">
+        <span className="EditorTitle">{label}</span>
+        {toolBarItems &&
+          toolBarItems.map((item, i) => (
+            <ToolbarItemComponent key={i} item={item} />
+          ))}
+      </div>
+    </div>
+  );
+};
+
+const ToolbarItemComponent: FunctionComponent<{ item: ToolbarItem }> = ({
+  item,
+}) => {
+  if (item.type === "button") {
+    const { onClick, color, label, tooltip, icon } = item;
+    if (icon) {
+      return (
+        <span className="EditorToolbarItem" style={{ color }}>
+          <IconButton
+            onClick={onClick}
+            disabled={!onClick}
+            color="inherit"
+            size="small"
+            title={tooltip}
+          >
+            {icon}
+            {label && <span className="ToolbarButtonText">{label}</span>}
+          </IconButton>
+        </span>
+      );
+    } else {
+      return (
+        <span className="EditorToolbarItem">
+          <Link
+            onClick={onClick}
+            color={color || "gray"}
+            component="button"
+            underline="none"
+            title={tooltip}
+          >
+            {label}
+          </Link>
+          &nbsp;&nbsp;&nbsp;
+        </span>
+      );
+    }
+  } else if (item.type === "text") {
+    return (
+      <span
+        className="EditorToolbarItem"
+        style={{ color: item.color || "gray" }}
+        title={item.label}
+      >
+        {item.label}&nbsp;&nbsp;&nbsp;
+      </span>
+    );
+  } else {
+    return <span>unknown toolbar item type</span>;
+  }
+};
diff --git a/gui/src/app/Scripting/ScriptEditor.tsx b/gui/src/app/Scripting/ScriptEditor.tsx
index 97ec153f..b96c4960 100644
--- a/gui/src/app/Scripting/ScriptEditor.tsx
+++ b/gui/src/app/Scripting/ScriptEditor.tsx
@@ -1,6 +1,7 @@
 import { Help, PlayArrow } from "@mui/icons-material";
 import { SplitDirection, Splitter } from "@SpComponents/Splitter";
-import TextEditor, { ToolbarItem } from "@SpComponents/TextEditor";
+import TextEditor from "@SpComponents/TextEditor";
+import { ToolbarItem } from "@SpComponents/ToolBar";
 import { FileNames } from "@SpCore/FileMapping";
 import { ProjectContext } from "@SpCore/ProjectContextProvider";
 import { ProjectKnownFiles } from "@SpCore/ProjectDataModel";
diff --git a/gui/src/localStyles.css b/gui/src/localStyles.css
index daf5216f..1654f6cd 100644
--- a/gui/src/localStyles.css
+++ b/gui/src/localStyles.css
@@ -116,12 +116,15 @@ span.EditorToolbarItem {
   letter-spacing: normal;
   display: inline-block;
   white-space: normal;
+  padding-left: 5px;
+  font-size: 14px;
 }
 
 @container EditorMenuBar (max-width: 500px) {
   .ToolbarButtonText {
     display: none;
     width: 0;
+    padding: 0;
   }
   span.EditorToolbarItem {
     padding: 0px;