From 06dbe8cc069a19761d7e2388bfe3af65c977d45d Mon Sep 17 00:00:00 2001
From: James Brunton <james@stirlingpdf.com>
Date: Mon, 15 Sep 2025 15:00:33 +0100
Subject: [PATCH] Improve tool step titles

---
 frontend/public/locales/en-GB/translation.json      | 13 ++++++-------
 .../changeMetadata/steps/AdvancedOptionsStep.tsx    |  1 -
 .../tools/changeMetadata/steps/DeleteAllStep.tsx    |  3 +--
 .../components/tooltips/useChangeMetadataTips.ts    |  8 ++------
 frontend/src/tools/ChangeMetadata.tsx               |  6 +++---
 5 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/frontend/public/locales/en-GB/translation.json b/frontend/public/locales/en-GB/translation.json
index 8d2456fb3..b2aa2ba23 100644
--- a/frontend/public/locales/en-GB/translation.json
+++ b/frontend/public/locales/en-GB/translation.json
@@ -1087,11 +1087,11 @@
       "title": "Metadata Settings"
     },
     "standardFields": {
-      "title": "Standard Metadata"
+      "title": "Standard Fields"
     },
     "deleteAll": {
-      "label": "Delete all metadata",
-      "description": "Remove all metadata from the PDF document"
+      "label": "Remove Existing Metadata",
+      "checkbox": "Delete all metadata"
     },
     "title": {
       "label": "Title",
@@ -1118,7 +1118,7 @@
       "placeholder": "Document producer"
     },
     "dates": {
-      "title": "Document Dates"
+      "title": "Date Fields"
     },
     "creationDate": {
       "label": "Creation Date",
@@ -1130,7 +1130,6 @@
     },
     "trapped": {
       "label": "Trapped Status",
-      "description": "Indicates whether the document has been trapped for high-quality printing",
       "unknown": "Unknown",
       "true": "True",
       "false": "False"
@@ -1181,8 +1180,8 @@
         "bullet3": "Delete All: Remove all metadata for privacy"
       },
       "deleteAll": {
-        "title": "Delete All Metadata",
-        "text": "Complete metadata removal for privacy and clean documents."
+        "title": "Remove Existing Metadata",
+        "text": "Complete metadata deletion to ensure privacy."
       },
       "customFields": {
         "title": "Custom Metadata",
diff --git a/frontend/src/components/tools/changeMetadata/steps/AdvancedOptionsStep.tsx b/frontend/src/components/tools/changeMetadata/steps/AdvancedOptionsStep.tsx
index 8d29b8e49..0edffe21d 100644
--- a/frontend/src/components/tools/changeMetadata/steps/AdvancedOptionsStep.tsx
+++ b/frontend/src/components/tools/changeMetadata/steps/AdvancedOptionsStep.tsx
@@ -28,7 +28,6 @@ const AdvancedOptionsStep = ({
       {/* Trapped Status */}
       <Select
         label={t('changeMetadata.trapped.label', 'Trapped Status')}
-        description={t('changeMetadata.trapped.description', 'Indicates whether the document has been trapped for high-quality printing')}
         value={parameters.trapped}
         onChange={(value) => {
           if (value) {
diff --git a/frontend/src/components/tools/changeMetadata/steps/DeleteAllStep.tsx b/frontend/src/components/tools/changeMetadata/steps/DeleteAllStep.tsx
index e3befb771..fbd983a18 100644
--- a/frontend/src/components/tools/changeMetadata/steps/DeleteAllStep.tsx
+++ b/frontend/src/components/tools/changeMetadata/steps/DeleteAllStep.tsx
@@ -17,8 +17,7 @@ const DeleteAllStep = ({
 
   return (
     <Checkbox
-      label={t('changeMetadata.deleteAll.label', 'Delete all metadata')}
-      description={t('changeMetadata.deleteAll.description', 'Remove all metadata from the PDF document')}
+      label={t('changeMetadata.deleteAll.checkbox', 'Delete all metadata')}
       checked={parameters.deleteAll}
       onChange={(e) => onParameterChange('deleteAll', e.target.checked)}
       disabled={disabled}
diff --git a/frontend/src/components/tooltips/useChangeMetadataTips.ts b/frontend/src/components/tooltips/useChangeMetadataTips.ts
index b7357e0ae..3b8618955 100644
--- a/frontend/src/components/tooltips/useChangeMetadataTips.ts
+++ b/frontend/src/components/tooltips/useChangeMetadataTips.ts
@@ -6,12 +6,11 @@ export const useDeleteAllTips = (): TooltipContent => {
 
   return {
     header: {
-      title: t("changeMetadata.tooltip.deleteAll.title", "Delete All Metadata")
+      title: t("changeMetadata.tooltip.deleteAll.title", "Remove Existing Metadata")
     },
     tips: [
       {
-        title: t("changeMetadata.tooltip.deleteAll.title", "Delete All Metadata"),
-        description: t("changeMetadata.tooltip.deleteAll.text", "Complete metadata removal for privacy and clean documents."),
+        description: t("changeMetadata.tooltip.deleteAll.text", "Complete metadata deletion to ensure privacy."),
       }
     ]
   };
@@ -26,7 +25,6 @@ export const useStandardMetadataTips = (): TooltipContent => {
     },
     tips: [
       {
-        title: t("changeMetadata.tooltip.standardFields.title", "Standard Fields"),
         description: t("changeMetadata.tooltip.standardFields.text", "Common PDF metadata fields that describe the document."),
         bullets: [
           t("changeMetadata.tooltip.standardFields.bullet1", "Title: Document name or heading"),
@@ -49,7 +47,6 @@ export const useDocumentDatesTips = (): TooltipContent => {
     },
     tips: [
       {
-        title: t("changeMetadata.tooltip.dates.title", "Date Fields"),
         description: t("changeMetadata.tooltip.dates.text", "When the document was created and modified."),
         bullets: [
           t("changeMetadata.tooltip.dates.bullet1", "Creation Date: When original document was made"),
@@ -69,7 +66,6 @@ export const useCustomMetadataTips = (): TooltipContent => {
     },
     tips: [
       {
-        title: t("changeMetadata.tooltip.customFields.title", "Custom Metadata"),
         description: t("changeMetadata.tooltip.customFields.text", "Add your own custom key-value metadata pairs."),
         bullets: [
           t("changeMetadata.tooltip.customFields.bullet1", "Add any custom fields relevant to your document"),
diff --git a/frontend/src/tools/ChangeMetadata.tsx b/frontend/src/tools/ChangeMetadata.tsx
index 2eb07afc7..a8f103ccf 100644
--- a/frontend/src/tools/ChangeMetadata.tsx
+++ b/frontend/src/tools/ChangeMetadata.tsx
@@ -65,7 +65,7 @@ const ChangeMetadata = (props: BaseToolProps) => {
 
   // Create step objects
   const createStandardMetadataStep = () => ({
-    title: t("changeMetadata.standardFields.title", "Standard Metadata"),
+    title: t("changeMetadata.standardFields.title", "Standard Fields"),
     isCollapsed: getActualCollapsedState(MetadataStep.STANDARD_METADATA),
     onCollapsedClick: () => handleStepToggle(MetadataStep.STANDARD_METADATA),
     tooltip: standardMetadataTips,
@@ -79,7 +79,7 @@ const ChangeMetadata = (props: BaseToolProps) => {
   });
 
   const createDocumentDatesStep = () => ({
-    title: t("changeMetadata.dates.title", "Document Dates"),
+    title: t("changeMetadata.dates.title", "Date Fields"),
     isCollapsed: getActualCollapsedState(MetadataStep.DOCUMENT_DATES),
     onCollapsedClick: () => handleStepToggle(MetadataStep.DOCUMENT_DATES),
     tooltip: documentDatesTips,
@@ -113,7 +113,7 @@ const ChangeMetadata = (props: BaseToolProps) => {
   const buildSteps = () => {
     const steps = [
       {
-        title: t("changeMetadata.deleteAll.label", "Delete All Metadata"),
+        title: t("changeMetadata.deleteAll.label", "Remove Existing Metadata"),
         isCollapsed: getActualCollapsedState(MetadataStep.DELETE_ALL),
         onCollapsedClick: () => handleStepToggle(MetadataStep.DELETE_ALL),
         tooltip: deleteAllTips,