Merge pull request #85 from wpmudev/fix/HUS-1337

🐛 fix(slide in): close button not working in safari mobile view.
wpmudev · Jun 9, 2023 · 779e02a · 779e02a
2 parents ff9c0d6 + cf9e741
commit 779e02a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/library/js/slidein-layouts.js b/library/js/slidein-layouts.js
@@ -12,6 +12,7 @@
 		const slidein = $( el );
 		const wrapper = slidein.find( '.hustle-slidein-content' );
 		const content = slidein.find( '.hustle-slidein-content > div' );
+		const maincontent = slidein.find( '.hustle-main-wrapper' );
 
 		let header = slidein.find( '.hustle-layout-header' );
 		let footer = slidein.find( '.hustle-layout-footer' );
@@ -63,7 +64,7 @@
 					'padding-top': headerHeight() + 'px'
 				});
 
-				content.css({
+				maincontent.css({
 					'max-height': 'calc(100vh - ' + ( footerHeight() + headerHeight() ) + 'px)'
 				});
 
@@ -72,7 +73,7 @@
 				});
 			} else {
 
-				content.css({
+				maincontent.css({
 					'max-height': 'calc(100vh - ' + ( footerHeight() + closeHeight() ) + 'px)'
 				});
 			}

diff --git a/library/scss/hustle-slidein.scss b/library/scss/hustle-slidein.scss
@@ -163,6 +163,10 @@
 			padding: 0;
 			overflow: auto;
 		}
+
+		.hustle-main-wrapper {
+			overflow-y: scroll;
+		}
 	}
 
 	// Correctly position slide-in.