Add Missing Theme Tab Labels

tone-row · Jan 12, 2025 · 5d9321d · 5d9321d
1 parent c0405ed
commit 5d9321d
Show file tree

Hide file tree

Showing 21 changed files with 14,550 additions and 19,590 deletions.
diff --git a/app/scripts/autotranslations.mjs b/app/scripts/autotranslations.mjs
@@ -1,7 +1,9 @@
+import { Configuration, OpenAIApi } from "openai";
+
 import dotenv from "dotenv";
 import fs from "fs";
-import { Configuration, OpenAIApi } from "openai";
 import path from "path";
+
 dotenv.config("./env");
 
 // Between Calls To Avoid Rate Limiting
@@ -124,7 +126,7 @@ for (const locale of locales) {
     /** pop off the first 6 phrases  */
     const batch = phrases.splice(0, 6);
 
-    const prompt = `Translate phrases from en to ${locale}\n\nEN\n${batch
+    const prompt = `Translate the following UI text strings for a flowcharting app from English to ${locale}.\n\nEN\n${batch
       .map((phrase) => `- ${phrase.text}`)
       .join("\n")}\n\n${locale.toUpperCase()}\n`;
 

diff --git a/app/src/components/RenameButton.tsx b/app/src/components/RenameButton.tsx
@@ -121,11 +121,13 @@ export const RenameButton = memo(function RenameButton({
                 <Trans>Rename</Trans>
               </SectionTitle>
               {hasProAccess && !isHosted ? (
-                <label className="flex gap-2 item-center mt-2">
-                  <span className="text-xs opacity-80">
-                    <Trans>Convert to hosted chart?</Trans>
-                  </span>
+                <label
+                  htmlFor="convert-to-hosted"
+                  className="flex gap-2 items-center mt-2 text-xs opacity-80"
+                >
+                  <Trans>Convert to hosted chart?</Trans>
                   <input
+                    id="convert-to-hosted"
                     type="checkbox"
                     checked={convertToHosted}
                     onChange={(e) => {

diff --git a/app/src/components/Tabs/ThemeTab.tsx b/app/src/components/Tabs/ThemeTab.tsx
@@ -303,7 +303,19 @@ const Form = createForm<{
         },
         {
           label: t`Text Max Width`,
-          title: () => <Article className="w-5 h-5" />,
+          title: () => (
+            <div className="grid gap-1">
+              <div className="flex items-center gap-2">
+                <Article className="w-5 h-5" />
+                <span className="font-semibold">
+                  <Trans>Text Max Width</Trans>
+                </span>
+              </div>
+              <span className="text-xs text-neutral-500 font-normal">
+                <Trans>Maximum width of text inside nodes</Trans>
+              </span>
+            </div>
+          ),
           id: "textMaxWidth",
           control: "range",
           value(data) {
@@ -348,7 +360,18 @@ const Form = createForm<{
         },
         {
           label: t`Text Horizontal Offset`,
-          title: () => t`Set Fixed Node Height`,
+          title: () => (
+            <div className="grid gap-1">
+              <div className="flex items-center gap-2">
+                <span className="font-semibold">
+                  <Trans>Fixed Node Height</Trans>
+                </span>
+              </div>
+              <span className="text-xs text-neutral-500 font-normal">
+                <Trans>Enable to set a consistent height for all nodes</Trans>
+              </span>
+            </div>
+          ),
           id: "useFixedHeight",
           control: "checkbox",
           value(data) {
@@ -360,7 +383,19 @@ const Form = createForm<{
         },
         {
           label: t`Fixed Node Height`,
-          title: () => <ArrowsOutLineVertical className="w-5 h-5" />,
+          title: () => (
+            <div className="grid gap-1">
+              <div className="flex items-center gap-2">
+                <ArrowsOutLineVertical className="w-5 h-5" />
+                <span className="font-semibold">
+                  <Trans>Fixed Height</Trans>
+                </span>
+              </div>
+              <span className="text-xs text-neutral-500 font-normal">
+                <Trans>Set a consistent height for all nodes</Trans>
+              </span>
+            </div>
+          ),
           id: "fixedHeight",
           control: "range",
           hidden(data) {
@@ -389,7 +424,19 @@ const Form = createForm<{
       elements: [
         {
           label: t`Curve Style`,
-          title: () => <LineSegment className="w-5 h-5" />,
+          title: () => (
+            <div className="grid gap-1">
+              <div className="flex items-center gap-2">
+                <LineSegment className="w-5 h-5" />
+                <span className="font-semibold">
+                  <Trans>Edge Style</Trans>
+                </span>
+              </div>
+              <span className="text-xs text-neutral-500 font-normal">
+                <Trans>Choose how edges connect between nodes</Trans>
+              </span>
+            </div>
+          ),
           id: "curveStyle",
           control: "select",
           value(data) {

diff --git a/app/src/lib/queries.ts b/app/src/lib/queries.ts
@@ -298,6 +298,7 @@ export async function makeChartPublic(id: number, isPublic: boolean) {
       message: "Duplicate key value violates unique constraint",
       details: "",
       hint: "",
+      name: "PostgrestError",
     };
     // If unique violation, generate a new public id
     let result;

diff --git a/app/src/locales/de/messages.js b/app/src/locales/de/messages.js