add/2793: Only take padding into account when using the anchor

WordPress · Nov 2, 2017 · bbae623 · bbae623
1 parent 20fa7fb
commit bbae623
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/components/popover/index.js b/components/popover/index.js
@@ -144,6 +144,7 @@ class Popover extends Component {
 		const isLeft = 'left' === xAxis;
 		const isRight = 'right' === xAxis;
 		let rect;
+		let topOffset = 0;
 		if ( range ) {
 			const rects = range.getClientRects();
 			if ( isLeft ) {
@@ -155,12 +156,12 @@ class Popover extends Component {
 			}
 		} else {
 			rect = anchor.parentNode.getBoundingClientRect();
-		}
-		// Offset top positioning by padding
-		const { paddingTop, paddingBottom } = window.getComputedStyle( anchor.parentNode );
-		let topOffset = parseInt( isTop ? paddingTop : paddingBottom, 10 );
-		if ( ! isTop ) {
-			topOffset *= -1;
+			// Offset top positioning by padding
+			const { paddingTop, paddingBottom } = window.getComputedStyle( anchor.parentNode );
+			topOffset = parseInt( isTop ? paddingTop : paddingBottom, 10 );
+			if ( ! isTop ) {
+				topOffset *= -1;
+			}
 		}
 
 		if ( isRight ) {