Remove required attribute from title input in printing widget (#317, #…

…318)
open-pioneer · May 22, 2024 · c927f60 · c927f60
1 parent 5623b5e
commit c927f60
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/.changeset/four-candles-greet.md b/.changeset/four-candles-greet.md
@@ -0,0 +1,5 @@
+---
+"@open-pioneer/printing": patch
+---
+
+Remove 'required' attribute from title input since it is not actually required
diff --git a/src/packages/printing/Printing.tsx b/src/packages/printing/Printing.tsx
@@ -98,7 +98,6 @@ export const Printing: FC<PrintingProps> = (props) => {
                         onChange={(event) => {
                             setTitle(event.target.value);
                         }}
-                        isRequired
                         autoFocus // eslint-disable-line jsx-a11y/no-autofocus
                     />
                 </HStack>

diff --git a/src/packages/printing/__snapshots__/Printing.test.tsx.snap b/src/packages/printing/__snapshots__/Printing.test.tsx.snap
@@ -26,12 +26,10 @@ exports[`should successfully create a printing component 1`] = `
         title
       </label>
       <input
-        aria-required="true"
         class="chakra-input css-1cjy4zv"
         data-theme="light"
         id="field-:r0:"
         placeholder="input.placeholder"
-        required=""
         value=""
       />
     </div>