From 13a8a2dbe241cd902e6a09fedc39e050f8ea0a7f Mon Sep 17 00:00:00 2001
From: Roman Glombovsky <rglombovsky@ebay.com>
Date: Thu, 28 Sep 2023 12:17:52 +0100
Subject: [PATCH] fix: replace 'value' with 'defaultValue' of the hidden select
 in Listbox component to remove a warning since it is a read only field.

---
 .../__snapshots__/index.spec.tsx.snap         | 20 +++++++++----------
 src/ebay-listbox-button/listbox-button.tsx    |  2 +-
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/ebay-listbox-button/__tests__/__snapshots__/index.spec.tsx.snap b/src/ebay-listbox-button/__tests__/__snapshots__/index.spec.tsx.snap
index 9b698828..d228e23e 100644
--- a/src/ebay-listbox-button/__tests__/__snapshots__/index.spec.tsx.snap
+++ b/src/ebay-listbox-button/__tests__/__snapshots__/index.spec.tsx.snap
@@ -35,8 +35,8 @@ exports[`Storyshots ebay-listbox-button Borderless 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
@@ -87,8 +87,8 @@ exports[`Storyshots ebay-listbox-button Default - no selected option 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="AA"
     hidden={true}
-    value="AA"
   >
     <option
       value="AA"
@@ -148,8 +148,8 @@ Array [
     </button>
     <select
       className="listbox-button__native"
+      defaultValue="BB"
       hidden={true}
-      value="BB"
     >
       <option
         value="AA"
@@ -200,8 +200,8 @@ exports[`Storyshots ebay-listbox-button Default 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
@@ -252,8 +252,8 @@ exports[`Storyshots ebay-listbox-button Disabled State 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
@@ -309,8 +309,8 @@ exports[`Storyshots ebay-listbox-button Floating label 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="AA"
     hidden={true}
-    value="AA"
   >
     <option
       value="AA"
@@ -360,8 +360,8 @@ exports[`Storyshots ebay-listbox-button Fluid 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
@@ -412,8 +412,8 @@ exports[`Storyshots ebay-listbox-button Invalid State 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
@@ -464,8 +464,8 @@ exports[`Storyshots ebay-listbox-button Statefull component 1`] = `
     </button>
     <select
       className="listbox-button__native"
+      defaultValue="California"
       hidden={true}
-      value="California"
     >
       <option
         value="California"
@@ -540,8 +540,8 @@ exports[`Storyshots ebay-listbox-button Too many options 1`] = `
   </button>
   <select
     className="listbox-button__native"
+    defaultValue="BB"
     hidden={true}
-    value="BB"
   >
     <option
       value="AA"
diff --git a/src/ebay-listbox-button/listbox-button.tsx b/src/ebay-listbox-button/listbox-button.tsx
index d3ab9151..906a0bfa 100644
--- a/src/ebay-listbox-button/listbox-button.tsx
+++ b/src/ebay-listbox-button/listbox-button.tsx
@@ -286,7 +286,7 @@ const ListboxButton: FC<EbayListboxButtonProps> = ({
                         {updatelistBoxButtonOptions}
                     </div>
                 </div>}
-            <select hidden className="listbox-button__native" name={name} value={selectedOption.props.value}>
+            <select hidden className="listbox-button__native" name={name} defaultValue={selectedOption.props.value}>
                 {
                     updatelistBoxButtonOptions.map((option, i) =>
                         <option value={option.props.value} key={i} />)