Update

tailwindlabs · May 3, 2024 · e13ba0c · e13ba0c
1 parent 19abeac
commit e13ba0c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/packages/@headlessui-react/src/hooks/document-overflow/handle-ios-locking.ts b/packages/@headlessui-react/src/hooks/document-overflow/handle-ios-locking.ts
@@ -94,13 +94,13 @@ export function handleIOSLocking(): ScrollLockStep<ContainerMetadata> {
           (e) => {
             // Check if we are scrolling inside any of the allowed containers, if not let's cancel the event!
             if (e.target instanceof HTMLElement) {
-              if (inAllowedContainer(e.target as HTMLElement)) {
-                // Some inputs like `<input type=range>` use touch events to
-                // allow interaction. We should not prevent this event.
-                if (e.target.tagName === 'INPUT') {
-                  return
-                }
+              // Some inputs like `<input type=range>` use touch events to
+              // allow interaction. We should not prevent this event.
+              if (e.target.tagName === 'INPUT') {
+                return
+              }
 
+              if (inAllowedContainer(e.target as HTMLElement)) {
                 // Even if we are in an allowed container, on iOS the main page can still scroll, we
                 // have to make sure that we `event.preventDefault()` this event to prevent that.
                 //

diff --git a/packages/@headlessui-vue/src/hooks/document-overflow/handle-ios-locking.ts b/packages/@headlessui-vue/src/hooks/document-overflow/handle-ios-locking.ts
@@ -94,13 +94,13 @@ export function handleIOSLocking(): ScrollLockStep<ContainerMetadata> {
           (e) => {
             // Check if we are scrolling inside any of the allowed containers, if not let's cancel the event!
             if (e.target instanceof HTMLElement) {
-              if (inAllowedContainer(e.target as HTMLElement)) {
-                // Some inputs like `<input type=range>` use touch events to
-                // allow interaction. We should not prevent this event.
-                if (e.target.tagName === 'INPUT') {
-                  return
-                }
+              // Some inputs like `<input type=range>` use touch events to
+              // allow interaction. We should not prevent this event.
+              if (e.target.tagName === 'INPUT') {
+                return
+              }
 
+              if (inAllowedContainer(e.target as HTMLElement)) {
                 // Even if we are in an allowed container, on iOS the main page can still scroll, we
                 // have to make sure that we `event.preventDefault()` this event to prevent that.
                 //