From 2123351fc1d1c529046c5179171e4d442ca20f55 Mon Sep 17 00:00:00 2001
From: Peter Stone <thepeterstone@gmail.com>
Date: Tue, 5 May 2026 12:30:50 -1000
Subject: fix: keep bottom nav visible when panning; expand drag handle touch
 target
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Bottom nav: remove it from the following/panning fade cycle — only
fabRecenter and fabRecordTrack swap; nav stays visible throughout.

Drag handle: 4dp × 36dp was too small and competed with BottomSheet
drag gestures. Now 40dp tall via inset drawable (visual bar unchanged,
touch area is a usable 40dp × 36dp).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt    | 4 ++--
 android-app/app/src/main/res/drawable/drag_handle_bg.xml         | 9 +++++++++
 android-app/app/src/main/res/layout/layout_instruments_sheet.xml | 7 +++----
 3 files changed, 14 insertions(+), 6 deletions(-)
 create mode 100644 android-app/app/src/main/res/drawable/drag_handle_bg.xml

(limited to 'android-app')

diff --git a/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt b/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
index 8821b7c..5b219c4 100644
--- a/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
+++ b/android-app/app/src/main/kotlin/org/terst/nav/MainActivity.kt
@@ -465,9 +465,9 @@ class MainActivity : AppCompatActivity(), SafetyFragment.SafetyListener {
                     mapCrosshair.visibility = if (following) View.GONE else View.VISIBLE
                     if (following) {
                         fadeOut(fabRecenter, gone = true)
-                        fadeIn(bottomNav, fabRecordTrack)
+                        fadeIn(fabRecordTrack)
                     } else {
-                        fadeOut(bottomNav, fabRecordTrack, gone = true)
+                        fadeOut(fabRecordTrack, gone = true)
                         fadeIn(fabRecenter)
                     }
                 }
diff --git a/android-app/app/src/main/res/drawable/drag_handle_bg.xml b/android-app/app/src/main/res/drawable/drag_handle_bg.xml
new file mode 100644
index 0000000..eea0325
--- /dev/null
+++ b/android-app/app/src/main/res/drawable/drag_handle_bg.xml
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="utf-8"?>
+<inset xmlns:android="http://schemas.android.com/apk/res/android"
+    android:insetTop="18dp"
+    android:insetBottom="18dp">
+    <shape android:shape="rectangle">
+        <solid android:color="@color/md_theme_outline" />
+        <corners android:radius="2dp" />
+    </shape>
+</inset>
diff --git a/android-app/app/src/main/res/layout/layout_instruments_sheet.xml b/android-app/app/src/main/res/layout/layout_instruments_sheet.xml
index aa785e6..59a92c3 100644
--- a/android-app/app/src/main/res/layout/layout_instruments_sheet.xml
+++ b/android-app/app/src/main/res/layout/layout_instruments_sheet.xml
@@ -12,13 +12,12 @@
     android:clickable="true"
     android:focusable="true">
 
-    <!-- Drag handle -->
+    <!-- Drag handle — visual bar is 4dp, touch target is 40dp via inset drawable -->
     <View
         android:id="@+id/drag_handle"
         android:layout_width="36dp"
-        android:layout_height="4dp"
-        android:layout_marginTop="12dp"
-        android:background="@color/md_theme_outline"
+        android:layout_height="40dp"
+        android:background="@drawable/drag_handle_bg"
         app:layout_constraintTop_toTopOf="parent"
         app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintEnd_toEndOf="parent" />
-- 
cgit v1.2.3