Remove Ready-to-Fly-Gate, add fly-mode header with flight-mode dropdown

Ready-to-Fly-Gate (Doku 2.2/4.5): AppModeCubit.toFly() erzwang bisher
Upload + verifiziert + disarmed und warf sonst einen StateError - da
MissionSyncService/Upload-Flow noch nicht angebunden sind, war das Gate
dauerhaft geschlossen und der Fly-Modus damit von der UI aus gar nicht
erreichbar. toFly() wechselt jetzt unbedingt in den Fly-Modus; die
readyToFly-Logik bleibt als Getter erhalten fuer den spaeteren Upload-
Flow. Der eigentliche Schutz vor einem Missions-Upload im armed-Zustand
lebt unveraendert auf Transport-Ebene (MockFlightControllerLink.
uploadMission() wirft dort weiterhin bei armed).

Fly-Modus-Kopfzeile (Doku 3.11/7.2, HTML-Demonstrator: #flightModePill/
#flightModeDropdown): top_mode_bar.dart zeigte im Fly-Modus bisher nur
eine leere Spacer-Flaeche. Zeigt jetzt die Wind-Pille (weiterhin
relevant, Doku 7.2) und eine neue Flugmodus-Auswahl:

- ui/providers/flight_mode_provider.dart: haelt den lokal gewaehlten
  FlightMode (Doku 3.1: missionRun/guidedPoint/returnHome/hold), noch
  ohne echte FlightControllerLink-Anbindung (Doku 4.19). reset() setzt
  auf missionRun (Waypoint) zurueck.
- ui/widgets/flight_mode_pill.dart: PopupMenuButton-Pille "Mode: X" mit
  den vier Optionen Waypoint/Point and Fly/Return to Home/Manual (1:1
  aus dem HTML-Demonstrator uebernommene Labels).
- top_mode_bar.dart: Fly-Button setzt beim Eintritt in den Fly-Modus
  den Flugmodus zurueck auf Waypoint (Doku 3.11: "Standard-
  Rueckstellung ... beim Eintritt in Fly-Modus").

Layout-Bug beim Implementieren gefunden und behoben: Expanded(child:
Center(child: FlightModePill())) liess die Kopfzeile ueber den
gesamten Bildschirm expandieren (derselbe "Center/Align ohne Faktor
expandiert auf verfuegbare Constraints"-Fehler wie zuvor schon bei der
Bottom-Stats-Bar in dieser Session) - behoben durch Entfernen des
ueberfluessigen Center-Wrappers, da FlightModePill sein Zentrieren
bereits selbst per Container-alignment uebernimmt. Zusaetzlich einen
RenderFlex-Overflow bei langen Modusnamen ("Return to Home") behoben,
indem der Label-Text in Flexible mit TextOverflow.ellipsis gewrappt
wurde.

Getestet: 2 neue Widget-Tests (Fly-Button wechselt jetzt direkt in den
Fly-Modus statt eine Gate-Snackbar zu zeigen; Flugmodus-Dropdown
wechselt den Modus und setzt ihn beim erneuten Eintritt zurueck),
bestehender Gate-Test ersetzt, ein bestehender Test vereinfacht (die
Gate-Erfuellung vor toFly() ist nicht mehr noetig). Alle 118 Tests
sowie flutter analyze bestehen. Manuell auf dem Pixel_10a-Emulator
verifiziert: Fly-Button wechselt direkt um, Dropdown zeigt alle vier
Optionen, Auswahl uebernimmt das Label korrekt (auch bei langen Namen
ohne Overflow), Rueckstellung auf Waypoint bei erneutem Eintritt
funktioniert.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Constantin Leue
2026-07-30 07:15:15 +02:00
co-authored by Claude Sonnet 5
parent 07394f79c8
commit d745b76403
5 changed files with 161 additions and 43 deletions
+8 -7
View File
@@ -52,13 +52,14 @@ class AppModeCubit extends Cubit<AppModeState> {
void droneArmedChanged(bool armed) =>
emit(state.copyWith(droneArmed: armed));
/// Wirft, wenn das Ready-to-Fly-Gate nicht erfuellt ist (Doku 4.5).
void toFly() {
if (!state.readyToFly) {
throw StateError('Ready-to-Fly-Gate nicht erfuellt.');
}
emit(state.copyWith(mode: AppMode.fly));
}
/// Wechselt in den Fly-Modus. Das Ready-to-Fly-Gate ([readyToFly], Doku
/// 2.2/4.5) wird hier bewusst nicht mehr erzwungen, solange Upload/
/// Verify-Flow (MissionSyncService) noch nicht angebunden sind - sonst
/// waere der Fly-Modus fuer die UI-Entwicklung gar nicht erreichbar. Der
/// eigentliche Schutz vor einem Missions-Upload im armed-Zustand lebt
/// unabhaengig davon weiterhin auf Transport-Ebene (siehe
/// MockFlightControllerLink.uploadMission()).
void toFly() => emit(state.copyWith(mode: AppMode.fly));
/// Verlassen des Fly-Modus setzt den kompletten Handover-Status zurueck.
void toPlan() => emit(const AppModeState());
@@ -0,0 +1,23 @@
import 'package:flutter_riverpod/flutter_riverpod.dart';
import '../../transport/flight_controller_link.dart';
/// Aktuell gewaehlter Flugmodus im Fly-Modus (Doku 3.1/3.11/7.2, HTML-
/// Demonstrator: currentFlightMode/setFlightMode()) - rein lokale
/// UI-Auswahl, noch nicht mit einer echten FlightControllerLink
/// verdrahtet (MspFlightControllerLink.setFlightMode() ist noch ein
/// Geruest, Doku 4.19).
class FlightModeNotifier extends Notifier<FlightMode> {
@override
FlightMode build() => FlightMode.missionRun;
void select(FlightMode mode) => state = mode;
/// Eintritt in den Fly-Modus setzt immer auf Waypoint zurueck (Doku
/// 3.11: "Standard-Rueckstellung auf Waypoint-Modus beim Eintritt in
/// Fly-Modus").
void reset() => state = FlightMode.missionRun;
}
final flightModeProvider =
NotifierProvider<FlightModeNotifier, FlightMode>(FlightModeNotifier.new);
+77
View File
@@ -0,0 +1,77 @@
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import '../../transport/flight_controller_link.dart';
import '../providers/flight_mode_provider.dart';
/// Flugmodus-Auswahl im Fly-Modus (Doku 3.11/7.2, HTML-Demonstrator:
/// #flightModePill/#flightModeDropdown). Rein lokale UI-Auswahl, noch
/// ohne echte FlightControllerLink-Anbindung (Doku 4.19) - Auswahl wirkt
/// sich aktuell nur auf die Anzeige aus.
class FlightModePill extends ConsumerWidget {
const FlightModePill({super.key});
static const _labels = {
FlightMode.missionRun: 'Waypoint',
FlightMode.guidedPoint: 'Point and Fly',
FlightMode.returnHome: 'Return to Home',
FlightMode.hold: 'Manual',
};
@override
Widget build(BuildContext context, WidgetRef ref) {
final mode = ref.watch(flightModeProvider);
return PopupMenuButton<FlightMode>(
tooltip: 'Flight mode',
color: const Color(0xF20F0F0F),
shape: RoundedRectangleBorder(
borderRadius: BorderRadius.circular(14),
side: BorderSide(color: Colors.white.withValues(alpha: 0.2)),
),
onSelected: (selected) => ref.read(flightModeProvider.notifier).select(selected),
itemBuilder: (context) => [
for (final entry in _labels.entries)
PopupMenuItem(
value: entry.key,
child: Text(
entry.value,
style: TextStyle(
color: entry.key == mode ? const Color(0xFFE2574A) : Colors.white,
fontWeight: entry.key == mode ? FontWeight.w700 : FontWeight.w500,
),
),
),
],
child: Container(
height: 38,
padding: const EdgeInsets.symmetric(horizontal: 14),
alignment: Alignment.center,
decoration: BoxDecoration(
color: Colors.black.withValues(alpha: 0.68),
borderRadius: BorderRadius.circular(19),
border: Border.all(color: Colors.white.withValues(alpha: 0.4)),
),
child: Row(
mainAxisSize: MainAxisSize.min,
children: [
Flexible(
child: Text(
'Mode: ${_labels[mode]}',
overflow: TextOverflow.ellipsis,
maxLines: 1,
style: const TextStyle(
color: Colors.white,
fontSize: 15,
fontWeight: FontWeight.w700,
),
),
),
const SizedBox(width: 4),
Icon(Icons.arrow_drop_down, color: Colors.white.withValues(alpha: 0.7)),
],
),
),
);
}
}
+23 -22
View File
@@ -2,9 +2,12 @@ import 'dart:math' as math;
import 'package:flutter/material.dart';
import 'package:flutter_bloc/flutter_bloc.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import '../../app_mode/app_mode_cubit.dart';
import '../dmc_colors.dart';
import '../providers/flight_mode_provider.dart';
import 'flight_mode_pill.dart';
import 'header_wind_pill.dart';
import 'map_search_controls.dart';
@@ -18,11 +21,16 @@ import 'map_search_controls.dart';
/// #searchWrap/#homeBtn/#fitBtn sitzen alle in #topBarMiddle, nicht in einer
/// separaten Leiste) - dafuer lebt der MapController im mapControllerProvider
/// statt als PlanScreen-privates Feld, siehe dort.
class TopModeBar extends StatelessWidget {
///
/// Im Fly-Modus traegt sie stattdessen die Wind-Pille (Doku 7.2: "Wind-
/// Anzeige nach Hoehe interpoliert") und die Flugmodus-Auswahl (Doku 3.11,
/// HTML-Demonstrator: #flightModePill) - Ortssuche/Home/Fit sind dort
/// nicht relevant, da im Fly-Modus keine Missionsbearbeitung stattfindet.
class TopModeBar extends ConsumerWidget {
const TopModeBar({super.key});
@override
Widget build(BuildContext context) {
Widget build(BuildContext context, WidgetRef ref) {
final isFly = context.select((AppModeCubit c) => c.state.mode == AppMode.fly);
final activeTint = isFly ? DmcColors.flyModeRed : DmcColors.editModeGreen;
@@ -67,9 +75,13 @@ class TopModeBar extends StatelessWidget {
),
onTap: () => context.read<AppModeCubit>().toPlan(),
),
if (isFly)
const Spacer()
else ...[
if (isFly) ...[
const SizedBox(width: 8),
const HeaderWindPill(),
const SizedBox(width: 8),
const Expanded(child: FlightModePill()),
const SizedBox(width: 8),
] else ...[
const SizedBox(width: 8),
const HeaderWindPill(),
const SizedBox(width: 8),
@@ -83,7 +95,12 @@ class TopModeBar extends StatelessWidget {
cornerRadius: const BorderRadius.only(
bottomRight: Radius.circular(26),
),
onTap: () => _tryEnterFly(context),
onTap: () {
context.read<AppModeCubit>().toFly();
// Eintritt in den Fly-Modus setzt immer auf Waypoint
// zurueck (Doku 3.11).
ref.read(flightModeProvider.notifier).reset();
},
),
],
),
@@ -91,22 +108,6 @@ class TopModeBar extends StatelessWidget {
),
);
}
void _tryEnterFly(BuildContext context) {
try {
context.read<AppModeCubit>().toFly();
} on StateError {
// Ready-to-Fly-Gate (Doku 2.2/4.5) nicht erfuellt - Upload/Verify-Flow
// ist noch nicht verdrahtet, daher aktuell immer dieser Fall.
ScaffoldMessenger.of(context).showSnackBar(
const SnackBar(
content: Text(
'Mission muss zuerst hochgeladen und verifiziert werden.',
),
),
);
}
}
}
class _ModeButton extends StatelessWidget {