Fix #4934: Password keydown called twice (#4935)

primefaces · Sep 20, 2023 · e1f1265 · e1f1265
1 parent 226603e
commit e1f1265
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 15 deletions.
diff --git a/components/lib/blockui/__snapshots__/BlockUI.spec.js.snap b/components/lib/blockui/__snapshots__/BlockUI.spec.js.snap
@@ -209,7 +209,7 @@ exports[`BlockUI container style and className 1`] = `
         </div>
       </div>
       <div
-        class="p-blockui p-component-overlay p-component-overlay-enter block-jest"
+        class="block-jest p-blockui p-component-overlay p-component-overlay-enter"
         data-pc-section="mask"
         style="cursor: move; position: absolute; top: 0px; left: 0px; width: 100%; height: 100%; z-index: 4205;"
       />

diff --git a/components/lib/password/Password.js b/components/lib/password/Password.js
@@ -380,7 +380,6 @@ export const Password = React.memo(
                 className: cx('root', { isFilled, focusedState }),
                 style: props.style
             },
-            PasswordBase.getOtherProps(props),
             ptm('root')
         );
 

diff --git a/components/lib/password/PasswordBase.js b/components/lib/password/PasswordBase.js
@@ -91,7 +91,7 @@ export const PasswordBase = ComponentBase.extend({
         panelClassName: null,
         transitionOptions: null,
         tabIndex: null,
-        value: null,
+        value: undefined,
         onInput: null,
         onShow: null,
         onHide: null,

diff --git a/components/lib/splitter/__snapshots__/Splitter.spec.js.snap b/components/lib/splitter/__snapshots__/Splitter.spec.js.snap
@@ -10,7 +10,7 @@ exports[`Splitter Nested 1`] = `
   >
     <div
       class="p-splitter-panel flex align-items-center justify-content-center"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(20% - 4px);"
@@ -29,7 +29,7 @@ exports[`Splitter Nested 1`] = `
     </div>
     <div
       class="p-splitter-panel p-splitter-panel-nested"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(80% - 4px);"
@@ -41,7 +41,7 @@ exports[`Splitter Nested 1`] = `
       >
         <div
           class="p-splitter-panel flex align-items-center justify-content-center"
-          data-pc-name="splitter"
+          data-pc-name=""
           data-pc-section="root"
           role="presentation"
           style="flex-basis: calc(15% - 4px);"
@@ -60,7 +60,7 @@ exports[`Splitter Nested 1`] = `
         </div>
         <div
           class="p-splitter-panel p-splitter-panel-nested"
-          data-pc-name="splitter"
+          data-pc-name=""
           data-pc-section="root"
           role="presentation"
           style="flex-basis: calc(85% - 4px);"
@@ -72,7 +72,7 @@ exports[`Splitter Nested 1`] = `
           >
             <div
               class="p-splitter-panel flex align-items-center justify-content-center"
-              data-pc-name="splitter"
+              data-pc-name=""
               data-pc-section="root"
               role="presentation"
               style="flex-basis: calc(20% - 4px);"
@@ -91,7 +91,7 @@ exports[`Splitter Nested 1`] = `
             </div>
             <div
               class="p-splitter-panel flex align-items-center justify-content-center"
-              data-pc-name="splitter"
+              data-pc-name=""
               data-pc-section="root"
               role="presentation"
               style="flex-basis: calc(80% - 4px);"
@@ -116,7 +116,7 @@ exports[`Splitter Single Panel with size 1`] = `
   >
     <div
       class="p-splitter-panel"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(5% - 0px);"
@@ -147,7 +147,7 @@ exports[`Splitter Single Panel without size 1`] = `
   >
     <div
       class="p-splitter-panel"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(100% - 0px);"
@@ -178,7 +178,7 @@ exports[`Splitter Splitter requires two SplitterPanel components to wrap. 1`] =
   >
     <div
       class="p-splitter-panel flex align-items-center justify-content-center"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(50% - 4px);"
@@ -197,7 +197,7 @@ exports[`Splitter Splitter requires two SplitterPanel components to wrap. 1`] =
     </div>
     <div
       class="p-splitter-panel flex align-items-center justify-content-center"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(50% - 4px);"
@@ -218,7 +218,7 @@ exports[`Splitter Vertical layout 1`] = `
   >
     <div
       class="p-splitter-panel"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(50% - 4px);"
@@ -237,7 +237,7 @@ exports[`Splitter Vertical layout 1`] = `
     </div>
     <div
       class="p-splitter-panel"
-      data-pc-name="splitter"
+      data-pc-name=""
       data-pc-section="root"
       role="presentation"
       style="flex-basis: calc(50% - 4px);"