Fix withoutPadding

WordPress · Oct 10, 2019 · dda3afd · dda3afd
1 parent 271d7e6
commit dda3afd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/packages/components/src/popover/index.js b/packages/components/src/popover/index.js
@@ -133,10 +133,10 @@ function withoutPadding( rect, element ) {
 		paddingLeft,
 		paddingRight,
 	} = window.getComputedStyle( element );
-	const top = parseInt( paddingTop, 10 );
-	const bottom = parseInt( paddingBottom, 10 );
-	const left = parseInt( paddingLeft, 10 );
-	const right = parseInt( paddingRight, 10 );
+	const top = paddingTop ? parseInt( paddingTop, 10 ) : 0;
+	const bottom = paddingBottom ? parseInt( paddingBottom, 10 ) : 0;
+	const left = paddingLeft ? parseInt( paddingLeft, 10 ) : 0;
+	const right = paddingRight ? parseInt( paddingRight, 10 ) : 0;
 
 	return {
 		x: rect.left + left,

diff --git a/packages/components/src/popover/test/__snapshots__/index.js.snap b/packages/components/src/popover/test/__snapshots__/index.js.snap
@@ -8,7 +8,7 @@ exports[`Popover should pass additional props to portaled element 1`] = `
     <div>
       <div>
         <div
-          class="components-popover is-bottom is-right components-animate__appear is-from-left is-from-top"
+          class="components-popover is-bottom is-center components-animate__appear is-from-top"
           role="tooltip"
           style=""
         >
@@ -33,7 +33,7 @@ exports[`Popover should render content 1`] = `
     <div>
       <div>
         <div
-          class="components-popover is-bottom is-right components-animate__appear is-from-left is-from-top"
+          class="components-popover is-bottom is-center components-animate__appear is-from-top"
           style=""
         >
           <div