From 49840a5ffef64342c13d07ed53826d195df319ba Mon Sep 17 00:00:00 2001
From: malmen237 <linda.malm@eyevinn.se>
Date: Fri, 8 Nov 2024 10:52:06 +0100
Subject: [PATCH] fix: added and updated some texts and added early return on
 clicking 'no layout'-selection

---
 src/components/modal/configureOutputModal/Decision.tsx | 10 ++++++++--
 .../multiviewLayoutSetup/MultiviewLayoutSetup.tsx      |  5 +++--
 .../production/multiviews/ProductionMultiviews.tsx     |  1 +
 src/i18n/locales/en.ts                                 |  1 +
 src/i18n/locales/sv.ts                                 |  1 +
 5 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/components/modal/configureOutputModal/Decision.tsx b/src/components/modal/configureOutputModal/Decision.tsx
index 426e173f..bf02ba1f 100644
--- a/src/components/modal/configureOutputModal/Decision.tsx
+++ b/src/components/modal/configureOutputModal/Decision.tsx
@@ -5,9 +5,15 @@ interface IDecision {
   onClose: () => void;
   onSave: () => void;
   className?: string;
+  buttonText?: string;
 }
 
-export default function Decision({ onClose, onSave, className }: IDecision) {
+export default function Decision({
+  onClose,
+  onSave,
+  className,
+  buttonText
+}: IDecision) {
   const t = useTranslate();
 
   return (
@@ -17,7 +23,7 @@ export default function Decision({ onClose, onSave, className }: IDecision) {
       }`}
     >
       <Button className="hover:bg-red-500" onClick={onClose} state="warning">
-        {t('close')}
+        {buttonText ? buttonText : t('close')}
       </Button>
       <Button
         className="relative flex hover:bg-green-400"
diff --git a/src/components/modal/multiviewLayoutSetup/MultiviewLayoutSetup.tsx b/src/components/modal/multiviewLayoutSetup/MultiviewLayoutSetup.tsx
index ae908b13..9b6778f3 100644
--- a/src/components/modal/multiviewLayoutSetup/MultiviewLayoutSetup.tsx
+++ b/src/components/modal/multiviewLayoutSetup/MultiviewLayoutSetup.tsx
@@ -151,6 +151,7 @@ export default function MultiviewLayoutSetup({
 
     switch (type) {
       case 'layout':
+        if (availableMultiviewLayouts.length === 0) return;
         setNewPresetName(name || '');
         if (chosenLayout) {
           setIsChecked(false);
@@ -169,7 +170,7 @@ export default function MultiviewLayoutSetup({
   };
 
   const handleChange = (viewId: string, value: string) => {
-    if (inputList && availableMultiviewLayouts) {
+    if (inputList) {
       const emptyView = {
         id: '',
         input_slot: 0,
@@ -253,7 +254,7 @@ export default function MultiviewLayoutSetup({
                     ? multiviewLayoutNames.map((singleItem) => ({
                         label: singleItem
                       }))
-                    : [{ label: 'No layouts available' }]
+                    : [{ label: t('preset.no_avaliable_layouts') }]
                 }
                 value={selectedMultiviewPreset?.name || ''}
                 update={(value) => handleLayoutUpdate(value, 'layout')}
diff --git a/src/components/production/multiviews/ProductionMultiviews.tsx b/src/components/production/multiviews/ProductionMultiviews.tsx
index efc5c29d..40b510cc 100644
--- a/src/components/production/multiviews/ProductionMultiviews.tsx
+++ b/src/components/production/multiviews/ProductionMultiviews.tsx
@@ -283,6 +283,7 @@ export default function ProductionMultiviews(props: ProductionMultiviewsProps) {
         <div className="flex flex-col">
           <Decision
             className="mt-6"
+            buttonText={t('clear')}
             onClose={() => clearInputs()}
             onSave={() => onSave()}
           />
diff --git a/src/i18n/locales/en.ts b/src/i18n/locales/en.ts
index f8a48d66..2d5443f4 100644
--- a/src/i18n/locales/en.ts
+++ b/src/i18n/locales/en.ts
@@ -702,6 +702,7 @@ export const en = {
     multiview_output_settings: 'Multiview output',
     select_multiview_layout: 'Layout',
     configure_layouts: 'Configure layouts',
+    no_avaliable_layouts: 'No available layouts',
     create_layout: 'Create new layout',
     update_layout: 'Update layout',
     no_updated_layout: 'No layout updated',
diff --git a/src/i18n/locales/sv.ts b/src/i18n/locales/sv.ts
index 4d5e341b..b4e7a8ad 100644
--- a/src/i18n/locales/sv.ts
+++ b/src/i18n/locales/sv.ts
@@ -710,6 +710,7 @@ export const sv = {
     no_multiview_found: 'Hittade ingen multiview',
     select_multiview_layout: 'Komposition',
     configure_layouts: 'Justera kompositioner',
+    no_avaliable_layouts: 'Inga kompositioner finns',
     create_layout: 'Skapa komposition',
     update_layout: 'Uppdatera komposition',
     no_updated_layout: 'Ingen uppdaterad komposition',