fix(focusTrap): allow to auto-resolve trap stack if was changed outside of controller

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

Author: Antreesy

src/components/NcModal/NcModal.vue

@@ -345,7 +345,6 @@ import Timer from '../../utils/Timer.js'
 import Close from 'vue-material-design-icons/Close.vue'
 import Pause from 'vue-material-design-icons/Pause.vue'
 import Play from 'vue-material-design-icons/Play.vue'
-import { useTrapStackControl } from '../../composables/useTrapStackControl.js'
 
 export default {
 	name: 'NcModal',
@@ -616,10 +615,6 @@ export default {
 		},
 	},
 
-	created() {
-		useTrapStackControl(() => this.showModal)
-	},
-
 	beforeMount() {
 		window.addEventListener('keydown', this.handleKeydown)
 	},

src/utils/focusTrap.ts

@@ -45,10 +45,14 @@ export function createTrapStackController() {
 		},
 		/**
 		 * Unpause the paused focus trap stack
+		 * If the actual stack is different from the paused one, there were changes
+		 * outside of this controller, so we assume it's self-regulated and do not unpause.
 		 */
 		unpause() {
-			for (const trap of pausedStack) {
-				trap.unpause()
+			if (pausedStack.length === getTrapStack().length) {
+				for (const trap of pausedStack) {
+					trap.unpause()
+				}
 			}
 			pausedStack = []
 		},