Merge pull request #41092 from nextcloud/fix/non-responsive-app-navig…

…ation-toggle

[stable25] Fix: non responsive app navigation toggle

core/src/init.js

@@ -140,44 +140,17 @@ export const initCore = () => {
 
 		$('#app-content').prepend('<div id="app-navigation-toggle" class="icon-menu" style="display:none" tabindex="0"></div>')
 
-		// keep track whether snapper is currently animating, and
-		// prevent to call open or close while that is the case
-		// to avoid duplicating events (snap.js doesn't check this)
-		let animating = false
-		snapper.on('animating', () => {
-			// we need this because the trigger button
-			// is also implicitly wired to close by snapper
-			animating = true
-		})
-		snapper.on('animated', () => {
-			animating = false
-		})
-		snapper.on('start', () => {
-			// we need this because dragging triggers that
-			animating = true
-		})
-		snapper.on('end', () => {
-			// we need this because dragging stop triggers that
-			animating = false
-		})
-
-		// These are necessary because calling open or close
-		// on snapper during an animation makes it trigger an
-		// unfinishable animation, which itself will continue
-		// triggering animating events and cause high CPU load,
-		//
-		// Ref https://github.com/jakiestfu/Snap.js/issues/216
 		const oldSnapperOpen = snapper.open
 		const oldSnapperClose = snapper.close
 		const _snapperOpen = () => {
-			if (animating || snapper.state().state !== 'closed') {
+			if (snapper.state().state !== 'closed') {
 				return
 			}
 			oldSnapperOpen('left')
 		}
 
 		const _snapperClose = () => {
-			if (animating || snapper.state().state === 'closed') {
+			if (snapper.state().state === 'closed') {
 				return
 			}
 			oldSnapperClose()