From c8990d4523c4fc9a3b13855f0e388ac856736901 Mon Sep 17 00:00:00 2001
From: Dallas <dallas.nicol@gmail.com>
Date: Tue, 21 Nov 2023 20:52:41 -0600
Subject: [PATCH 1/3] :ghost: use pf5 select for set-targets (#1526)

Also pulled the select out of the text-content wrapper which was
throwing off some styles.

---------

Signed-off-by: gitdallas <dallas.nicol@gmail.com>
Signed-off-by: Dallas <dallas.nicol@gmail.com>
Co-authored-by: Ian Bolton <ibolton@redhat.com>
---
 .../app/components/SimpleSelectTypeahead.tsx  | 15 ++++++--
 .../analysis-wizard/set-targets.tsx           | 37 +++++++++++--------
 2 files changed, 33 insertions(+), 19 deletions(-)

diff --git a/client/src/app/components/SimpleSelectTypeahead.tsx b/client/src/app/components/SimpleSelectTypeahead.tsx
index c442c5149d..3e61aea324 100644
--- a/client/src/app/components/SimpleSelectTypeahead.tsx
+++ b/client/src/app/components/SimpleSelectTypeahead.tsx
@@ -25,7 +25,9 @@ export interface ISimpleSelectBasicProps {
   toggleId?: string;
   toggleAriaLabel?: string;
   selectMultiple?: boolean;
+  width?: number;
   noResultsFoundText?: string;
+  hideClearButton?: boolean;
 }
 
 export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
@@ -37,7 +39,9 @@ export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
   toggleId,
   toggleAriaLabel,
   selectMultiple = false,
+  width,
   noResultsFoundText,
+  hideClearButton = false,
 }) => {
   const [isOpen, setIsOpen] = React.useState(false);
   const [selected, setSelected] = React.useState<string | string[]>(
@@ -190,7 +194,8 @@ export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
       variant="typeahead"
       onClick={onToggleClick}
       isExpanded={isOpen}
-      isFullWidth
+      isFullWidth={!width}
+      style={{ width: width && width + "px" }}
     >
       <TextInputGroup isPlain>
         <TextInputGroupMain
@@ -199,7 +204,9 @@ export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
           onChange={onTextInputChange}
           onKeyDown={onInputKeyDown}
           onBlur={() => {
-            setInputValue("");
+            selectMultiple
+              ? setInputValue("")
+              : setInputValue(selected.toString());
           }}
           id="typeahead-select-input"
           autoComplete="off"
@@ -243,7 +250,7 @@ export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
               <TimesIcon aria-hidden />
             </Button>
           )}
-          {!selectMultiple && !!inputValue && (
+          {!hideClearButton && !selectMultiple && !!inputValue && (
             <Button
               variant="plain"
               onClick={() => {
@@ -264,13 +271,13 @@ export const SimpleSelectTypeahead: React.FC<ISimpleSelectBasicProps> = ({
   );
   return (
     <>
-      <div>selected: {[selected].join(",")}</div>
       <Select
         id={id}
         isOpen={isOpen}
         selected={selected}
         onSelect={onSelect}
         onOpenChange={() => {
+          setFilterValue("");
           setIsOpen(false);
         }}
         toggle={toggle}
diff --git a/client/src/app/pages/applications/analysis-wizard/set-targets.tsx b/client/src/app/pages/applications/analysis-wizard/set-targets.tsx
index 58f12f2fd2..97e191d1a9 100644
--- a/client/src/app/pages/applications/analysis-wizard/set-targets.tsx
+++ b/client/src/app/pages/applications/analysis-wizard/set-targets.tsx
@@ -16,7 +16,7 @@ import { AnalysisWizardFormValues } from "./schema";
 import { useSetting } from "@app/queries/settings";
 import { useFetchTargets } from "@app/queries/targets";
 import { Target } from "@app/api/models";
-import { SimpleSelect } from "@app/components/SimpleSelect";
+import { SimpleSelectTypeahead } from "@app/components/SimpleSelectTypeahead";
 
 export const SetTargets: React.FC = () => {
   const { t } = useTranslation();
@@ -135,20 +135,28 @@ export const SetTargets: React.FC = () => {
           {t("wizard.terms.setTargets")}
         </Title>
         <Text>{t("wizard.label.setTargets")}</Text>
-        <SimpleSelect
-          width={200}
-          variant="typeahead"
-          id="action-select"
-          toggleId="action-select-toggle"
-          toggleAriaLabel="Action select dropdown toggle"
-          aria-label={"Select provider"}
-          value={provider}
-          options={["Java", "Go"]}
-          onChange={(selection) => {
-            setProvider(selection as string);
-          }}
-        />
       </TextContent>
+      <SimpleSelectTypeahead
+        width={200}
+        value={provider}
+        toggleAriaLabel="Action select dropdown toggle"
+        toggleId="action-select-toggle"
+        hideClearButton
+        id="action-select"
+        options={[
+          {
+            value: "Java",
+            children: "Java",
+          },
+          {
+            value: "Go",
+            children: "Go",
+          },
+        ]}
+        onChange={(selection) => {
+          setProvider(selection as string);
+        }}
+      />
       {values.selectedTargets.length === 0 &&
         values.customRulesFiles.length === 0 &&
         !values.sourceRepository && (
@@ -158,7 +166,6 @@ export const SetTargets: React.FC = () => {
             title={t("wizard.label.skipTargets")}
           />
         )}
-
       <Gallery hasGutter>
         {targetOrderSetting.isSuccess
           ? targetOrderSetting.data.map((id, index) => {

From 86c80d97e912b211c441dc03970644eee1224454 Mon Sep 17 00:00:00 2001
From: Ian Bolton <ibolton@redhat.com>
Date: Wed, 22 Nov 2023 09:06:44 -0500
Subject: [PATCH 2/3] :bug: Fix text for stakeholder group empty state (#1559)

Resolves https://issues.redhat.com/browse/MTA-1715

Signed-off-by: ibolton336 <ibolton@redhat.com>
---
 .../pages/controls/stakeholder-groups/stakeholder-groups.tsx  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/src/app/pages/controls/stakeholder-groups/stakeholder-groups.tsx b/client/src/app/pages/controls/stakeholder-groups/stakeholder-groups.tsx
index 9978549230..ae388d4b7f 100644
--- a/client/src/app/pages/controls/stakeholder-groups/stakeholder-groups.tsx
+++ b/client/src/app/pages/controls/stakeholder-groups/stakeholder-groups.tsx
@@ -240,12 +240,12 @@ export const StakeholderGroups: React.FC = () => {
                   <EmptyStateIcon icon={CubesIcon} />
                   <Title headingLevel="h2" size="lg">
                     {t("composed.noDataStateTitle", {
-                      what: t("terms.stakeholder").toLowerCase(),
+                      what: t("terms.stakeholderGroup").toLowerCase(),
                     })}
                   </Title>
                   <EmptyStateBody>
                     {t("composed.noDataStateBody", {
-                      what: t("terms.stakeholder").toLowerCase(),
+                      what: t("terms.stakeholderGroup").toLowerCase(),
                     })}
                   </EmptyStateBody>
                 </EmptyState>

From 63e27fda52fec8e1f91a1f6dcd75616b9c08c310 Mon Sep 17 00:00:00 2001
From: Ian Bolton <ibolton@redhat.com>
Date: Wed, 22 Nov 2023 09:11:53 -0500
Subject: [PATCH 3/3] :bug: Fix filter name for archetypes (#1560)

https://issues.redhat.com/browse/MTA-1711

Signed-off-by: ibolton336 <ibolton@redhat.com>
---
 .../applications/applications-table/applications-table.tsx  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/client/src/app/pages/applications/applications-table/applications-table.tsx b/client/src/app/pages/applications/applications-table/applications-table.tsx
index d5660fbca9..947bc656e3 100644
--- a/client/src/app/pages/applications/applications-table/applications-table.tsx
+++ b/client/src/app/pages/applications/applications-table/applications-table.tsx
@@ -340,12 +340,12 @@ export const ApplicationsTable: React.FC = () => {
         getItemValue: (item) => item?.name || "",
       },
       {
-        key: "archetype",
-        title: t("terms.archetype"),
+        key: "archetypes",
+        title: t("terms.archetypes"),
         type: FilterType.multiselect,
         placeholderText:
           t("actions.filterBy", {
-            what: t("terms.archetype").toLowerCase(),
+            what: t("terms.archetypes").toLowerCase(),
           }) + "...",
         getItemValue: (item) => {
           const archetypeNames = item?.archetypes