From 7d245ed99b23b2498cf3db42580fa9016e8fdd10 Mon Sep 17 00:00:00 2001
From: Ian Bolton <ibolton@redhat.com>
Date: Tue, 26 Mar 2024 13:57:08 -0400
Subject: [PATCH] PR suggestions

Signed-off-by: Ian Bolton <ibolton@redhat.com>
---
 client/public/locales/en/translation.json                 | 2 ++
 .../components/manage-columns-modal.tsx                   | 2 ++
 .../components/manage-columns-toolbar.tsx                 | 8 ++++----
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/client/public/locales/en/translation.json b/client/public/locales/en/translation.json
index 161c2e055f..379a37d354 100644
--- a/client/public/locales/en/translation.json
+++ b/client/public/locales/en/translation.json
@@ -132,6 +132,7 @@
       "import": "Import {{what}}",
       "leavePage": "Leave page",
       "leaveAssessment": "Leave assessment",
+      "manageColumns": "Manage columns",
       "new": "New {{what}}",
       "newArchetype": "Create new archetype",
       "newAssessment": "New assessment",
@@ -213,6 +214,7 @@
     "selectedBecauseArchetypeTags": "Selected because the archetype's tags include {{tags}}",
     "selectedBecauseAppOrArchetypeTags": "Selected because the application or archetype's tags include {{tags}}",
     "selectOwnerFromStakeholdersList": "Select owner from list of stakeholders",
+    "manageColumnsDescription": "Select the columns you want to display in the table view.",
     "suggestedAdoptionPlanHelpText": "The suggested approach to migration based on effort, priority, and dependencies.",
     "taskInProgressForTags": "A new analysis is in-progress.  Tags may be updated upon completion.",
     "toTagApplication": "Either no tags exist yet or you may not have permission to view any. If you have permission, try creating a new custom tag.",
diff --git a/client/src/app/pages/applications/applications-table/components/manage-columns-modal.tsx b/client/src/app/pages/applications/applications-table/components/manage-columns-modal.tsx
index d8aec5d7aa..ffbbb8dd33 100644
--- a/client/src/app/pages/applications/applications-table/components/manage-columns-modal.tsx
+++ b/client/src/app/pages/applications/applications-table/components/manage-columns-modal.tsx
@@ -82,6 +82,8 @@ export const ManageColumnsModal = <TColumnKey extends string>({
                 <DataListCheck
                   aria-labelledby={`check-${id}`}
                   checked={isVisible}
+                  //TODO: Dynamic disable logic based on idProperty definition in useTableControlState.
+                  // Currently, any column whose name is 'name' will not be allowed to be hidden
                   isDisabled={id === "name"}
                   onChange={(e, checked) => onSelect(id, checked)}
                 />
diff --git a/client/src/app/pages/applications/applications-table/components/manage-columns-toolbar.tsx b/client/src/app/pages/applications/applications-table/components/manage-columns-toolbar.tsx
index 91d4f5a726..7cfece0373 100644
--- a/client/src/app/pages/applications/applications-table/components/manage-columns-toolbar.tsx
+++ b/client/src/app/pages/applications/applications-table/components/manage-columns-toolbar.tsx
@@ -40,12 +40,12 @@ export const ManageColumnsToolbar = <TColumnKey extends string>({
       {isOpen && (
         <ManageColumnsModal
           onClose={() => setIsOpen(false)}
-          description={t("Selected columns will be displayed in the table.")}
+          description={t("message.manageColumnsDescription")}
           setColumns={setColumns}
           columns={columns}
-          saveLabel={t("Save")}
-          cancelLabel={t("Cancel")}
-          title={t("Manage Columns")}
+          saveLabel={t("actions.save")}
+          cancelLabel={t("actions.cancel")}
+          title={t("title.manageColumns")}
         />
       )}
     </>