ADFA-4320 Paper cut: Fix for swiping bottom sheet tabs incorrectly opening file menu (#1390)

* Fix for swiping bottom sheet tabs incorrectly opening file menu

* ADFA-4320 Skip drawer fling-open when swiping bottom sheet tabs

Tab-strip flings have enough horizontal velocity to trigger the editor
activity's drawer-open gesture detector, opening the file tree when the
user is just navigating between bottom-pane tabs.

In dispatchTouchEvent, check on ACTION_DOWN whether the touch falls
inside the bottom sheet TabLayout's global visible rect; if so, stop
forwarding the touch sequence to the gesture detector. The hamburger
button and the existing no-files horizontal-fling shortcut remain
untouched.

Reverts the bottom-sheet-state drawer lock from da1e435 — that
approach broke the hamburger because ActionBarDrawerToggle treats
LOCK_MODE_LOCKED_CLOSED as "do not open" and the lock also fired on
gestures outside the tab strip.

* ADFA-4320 Move tab-strip check into onFling and harden against teardown

Code review surfaced three issues with the previous implementation:

1. content.bottomSheet.binding.tabs reads a binding chain whose root
   getter throws IllegalStateException once _binding is cleared in
   preDestroy. Touches dispatched between preDestroy and window
   teardown would crash. Switched to contentOrNull, the file's existing
   defensive helper.

2. The dispatchTouchEvent flag was set only on ACTION_DOWN, so
   ACTION_POINTER_DOWN (multi-touch) could not re-evaluate it; a second
   finger could leak past or stay stuck-suppressed.

3. The check belonged with the other fling filters (startedNearTopEdge,
   noFilesOpen, etc.) in onFling, not in dispatchTouchEvent — and onFling
   already tracks the down event as e1, so no extra state is needed.

Folded the check into the isDrawerOpenFling branch using e1. Dropped
suppressDrawerGesture and bottomSheetTabsHitRect fields; restored
dispatchTouchEvent to its pre-PR form. The hit-test now runs only when
a drawer-open fling is actually detected, not on every ACTION_DOWN.

Author: hal-eisen-adfa

app/src/main/java/com/itsaky/androidide/activities/editor/BaseEditorActivity.kt

@@ -22,6 +22,7 @@ import android.content.Intent
 import android.content.ServiceConnection
 import android.content.res.Configuration
 import android.graphics.Color
+import android.graphics.Rect
 import android.graphics.drawable.GradientDrawable
 import android.os.Build
 import android.os.Bundle
@@ -1652,7 +1653,9 @@ abstract class BaseEditorActivity :
 						}
 
 						// Filter out diagonal flings so only an intentional right swipe opens the drawer.
-						if (isDrawerOpenFling) {
+						// A horizontal fling that started on the bottom-sheet tab strip is the user
+						// scrolling tabs, not asking for the drawer.
+						if (isDrawerOpenFling && !isTouchOnBottomSheetTabs(e1)) {
 							binding.editorDrawerLayout.openDrawer(GravityCompat.START)
 							return true
 						}
@@ -1672,6 +1675,13 @@ abstract class BaseEditorActivity :
 		return super.dispatchTouchEvent(ev)
 	}
 
+	private fun isTouchOnBottomSheetTabs(ev: MotionEvent): Boolean {
+		val tabs = contentOrNull?.bottomSheet?.binding?.tabs ?: return false
+		val rect = Rect()
+		if (!tabs.getGlobalVisibleRect(rect)) return false
+		return rect.contains(ev.rawX.toInt(), ev.rawY.toInt())
+	}
+
 	private fun showTooltip(tag: String) {
 		TooltipManager.showIdeCategoryTooltip(
 			context = this,