diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-03-23 07:07:17 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-03-23 07:07:17 +0000 |
| commit | 65c04f94ecd862ec2ba195e0128da84f79647ef1 (patch) | |
| tree | 3b6162172a8050ec9444a6f233321e6a3befe2c4 /android-app/app/src/main | |
| parent | 77892d354eda07d98e2dfa5d00fca1ed1f808d8b (diff) | |
fix: resolve Kotlin compilation errors from UI refactor
- Make InstrumentHandler.labelTrend and barometerTrendView nullable
- Remove anchorWatchHandler init block referencing non-existent view IDs
- Fix state.radiusM -> state.watchCircleRadiusMeters
- Add bottom_nav_weather_menu.xml with nav_forecast; point WeatherActivity at it
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'android-app/app/src/main')
4 files changed, 18 insertions, 16 deletions
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 bbc9853..d0b0e58 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 @@ -159,16 +159,7 @@ class MainActivity : AppCompatActivity(), SafetyFragment.SafetyListener { polarDiagramView = findViewById(R.id.polar_diagram_view) ) - anchorWatchHandler = AnchorWatchHandler( - context = this, - container = findViewById(R.id.anchor_config_container) ?: FrameLayout(this), // stub for now - statusText = findViewById(R.id.anchor_status_text) ?: android.widget.TextView(this), - radiusText = findViewById(R.id.anchor_radius_text) ?: android.widget.TextView(this), - buttonDecrease = findViewById(R.id.button_decrease_radius) ?: android.widget.Button(this), - buttonIncrease = findViewById(R.id.button_increase_radius) ?: android.widget.Button(this), - buttonSet = findViewById(R.id.button_set_anchor) ?: android.widget.Button(this), - buttonStop = findViewById(R.id.button_stop_anchor) ?: android.widget.Button(this) - ) + // anchorWatchHandler is initialized when the anchor config UI is available val mockPolarTable = createMockPolarTable() findViewById<PolarDiagramView>(R.id.polar_diagram_view).setPolarTable(mockPolarTable) @@ -242,7 +233,7 @@ class MainActivity : AppCompatActivity(), SafetyFragment.SafetyListener { } lifecycleScope.launch { LocationService.anchorWatchState.collect { state -> - safetyFragment.updateAnchorStatus(if (state.isActive) "Active: ${state.radiusM}m" else "Inactive") + safetyFragment.updateAnchorStatus(if (state.isActive) "Active: ${state.watchCircleRadiusMeters}m" else "Inactive") } } } diff --git a/android-app/app/src/main/kotlin/org/terst/nav/ui/InstrumentHandler.kt b/android-app/app/src/main/kotlin/org/terst/nav/ui/InstrumentHandler.kt index 63c6165..2f72153 100644 --- a/android-app/app/src/main/kotlin/org/terst/nav/ui/InstrumentHandler.kt +++ b/android-app/app/src/main/kotlin/org/terst/nav/ui/InstrumentHandler.kt @@ -21,8 +21,8 @@ class InstrumentHandler( private val valueDepth: TextView, private val valuePolarPct: TextView, private val valueBaro: TextView, - private val labelTrend: TextView, - private val barometerTrendView: BarometerTrendView, + private val labelTrend: TextView?, + private val barometerTrendView: BarometerTrendView?, private val polarDiagramView: PolarDiagramView ) { /** @@ -51,7 +51,7 @@ class InstrumentHandler( depth?.let { valueDepth.text = it } polarPct?.let { valuePolarPct.text = it } baro?.let { valueBaro.text = it } - trend?.let { labelTrend.text = it } + trend?.let { labelTrend?.text = it } } /** @@ -65,6 +65,6 @@ class InstrumentHandler( * Updates the barometer trend chart. */ fun updateBarometerTrend(history: List<BarometerReading>) { - barometerTrendView.setHistory(history) + barometerTrendView?.setHistory(history) } } diff --git a/android-app/app/src/main/res/layout/activity_weather.xml b/android-app/app/src/main/res/layout/activity_weather.xml index 36ea871..68374ca 100644 --- a/android-app/app/src/main/res/layout/activity_weather.xml +++ b/android-app/app/src/main/res/layout/activity_weather.xml @@ -14,7 +14,7 @@ android:id="@+id/bottom_nav" android:layout_width="match_parent" android:layout_height="wrap_content" - app:menu="@menu/bottom_nav_menu" + app:menu="@menu/bottom_nav_weather_menu" xmlns:app="http://schemas.android.com/apk/res-auto" /> </LinearLayout> diff --git a/android-app/app/src/main/res/menu/bottom_nav_weather_menu.xml b/android-app/app/src/main/res/menu/bottom_nav_weather_menu.xml new file mode 100644 index 0000000..d3c1427 --- /dev/null +++ b/android-app/app/src/main/res/menu/bottom_nav_weather_menu.xml @@ -0,0 +1,11 @@ +<?xml version="1.0" encoding="utf-8"?> +<menu xmlns:android="http://schemas.android.com/apk/res/android"> + <item + android:id="@+id/nav_map" + android:icon="@drawable/ic_map" + android:title="Map" /> + <item + android:id="@+id/nav_forecast" + android:icon="@drawable/ic_instruments" + android:title="Forecast" /> +</menu> |
