feat(04-02): add staleness detection to main speed display

- Add staleness check timer (1s interval) with proper disposal
- Speed/heading dim at 3s, show dashes at 10s (matching overlay)
- Apply staleness opacity to both speed and heading displays
- Remove WakelockPlus management (now in GpsDataManager)
- Remove WidgetsBindingObserver (now in GpsDataManager)
- Simplify background heartbeat (removed _isInBackground check)
- Wire overlay provider into GpsDataManager via initState
- Add haptic feedback to theme/unit cycling
- Update _buildSpeedDisplay to accept opacity parameter
This commit is contained in:
Mikkeli Matlock
2026-02-12 09:09:10 +09:00
parent 8bc1e392a5
commit 1dcfe60b4d
+81 -95
View File
@@ -4,10 +4,11 @@ import 'package:provider/provider.dart';
import 'package:permission_handler/permission_handler.dart'; import 'package:permission_handler/permission_handler.dart';
import 'dart:async'; import 'dart:async';
import 'dart:math' as math; import 'dart:math' as math;
import 'package:wakelock_plus/wakelock_plus.dart';
import '../config/timing_constants.dart'; import '../config/timing_constants.dart';
import '../config/overlay_constants.dart';
import '../config/color_themes.dart'; import '../config/color_themes.dart';
import '../services/gps_data_manager.dart'; import '../services/gps_data_manager.dart';
import '../models/processed_gps_data.dart';
import '../providers/settings_provider.dart'; import '../providers/settings_provider.dart';
import '../providers/overlay_provider.dart'; import '../providers/overlay_provider.dart';
import '../services/logger.dart'; import '../services/logger.dart';
@@ -19,31 +20,33 @@ class SpeedometerScreen extends StatefulWidget {
State<SpeedometerScreen> createState() => _SpeedometerScreenState(); State<SpeedometerScreen> createState() => _SpeedometerScreenState();
} }
class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindingObserver { class _SpeedometerScreenState extends State<SpeedometerScreen> {
// Local widget state only (not shared app state) // Local widget state only (not shared app state)
String _errorMessage = ''; String _errorMessage = '';
bool _isInBackground = false;
Timer? _backgroundHeartbeatTimer; Timer? _backgroundHeartbeatTimer;
Timer? _stalenessCheckTimer;
@override @override
void initState() { void initState() {
super.initState(); super.initState();
WidgetsBinding.instance.addObserver(this);
// Delay GPS init and error callback setup to after first frame so context is available // Delay GPS init and error callback setup to after first frame so context is available
WidgetsBinding.instance.addPostFrameCallback((_) { WidgetsBinding.instance.addPostFrameCallback((_) {
_initializeGpsManager(); _initializeGpsManager();
context.read<OverlayProvider>().onError = _handleOverlayError; context.read<OverlayProvider>().onError = _handleOverlayError;
// Wire overlay provider into GPS manager for lifecycle decisions
context.read<GpsDataManager>().setOverlayProvider(context.read<OverlayProvider>());
}); });
_enableWakelock();
_ensureOverlayPermission(); _ensureOverlayPermission();
_startBackgroundHeartbeat(); _startBackgroundHeartbeat();
_stalenessCheckTimer = Timer.periodic(OverlayConfig.STALENESS_CHECK_INTERVAL, (_) {
if (mounted) setState(() {}); // Trigger rebuild to re-evaluate staleness
});
} }
@override @override
void dispose() { void dispose() {
WidgetsBinding.instance.removeObserver(this);
_backgroundHeartbeatTimer?.cancel(); _backgroundHeartbeatTimer?.cancel();
WakelockPlus.disable(); _stalenessCheckTimer?.cancel();
try { try {
context.read<OverlayProvider>().onError = null; context.read<OverlayProvider>().onError = null;
} catch (e) { } catch (e) {
@@ -97,19 +100,6 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
); );
} }
Future<void> _enableWakelock() async {
try {
await WakelockPlus.enable();
final isEnabled = await WakelockPlus.enabled;
if (!isEnabled) {
Logger.warn('Wake lock not properly enabled, retrying...', 'Main');
await WakelockPlus.enable();
}
} catch (e) {
Logger.error('Wake lock failed: $e', 'Main');
}
}
Future<void> _ensureOverlayPermission() async { Future<void> _ensureOverlayPermission() async {
try { try {
// Note: Overlay permission request is handled by OverlayProvider, // Note: Overlay permission request is handled by OverlayProvider,
@@ -123,62 +113,20 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
_backgroundHeartbeatTimer = Timer.periodic(TimingConfig.HEARTBEAT_INTERVAL, (timer) { _backgroundHeartbeatTimer = Timer.periodic(TimingConfig.HEARTBEAT_INTERVAL, (timer) {
if (!mounted) return; if (!mounted) return;
final overlay = context.read<OverlayProvider>(); final overlay = context.read<OverlayProvider>();
if (_isInBackground && overlay.isOverlayActive) { if (overlay.isOverlayActive) {
overlay.pushCurrentData(); overlay.pushCurrentData();
} }
}); });
} }
@override Widget _buildSpeedDisplay(String speedText, ColorTheme currentTheme, double fontSize, double opacity) {
void didChangeAppLifecycleState(AppLifecycleState state) {
super.didChangeAppLifecycleState(state);
switch (state) {
case AppLifecycleState.paused:
if (!_isInBackground) {
_isInBackground = true;
_handleBackgroundTransition();
}
break;
case AppLifecycleState.resumed:
_isInBackground = false;
break;
case AppLifecycleState.inactive:
case AppLifecycleState.detached:
case AppLifecycleState.hidden:
break;
}
}
void _handleBackgroundTransition() {
_enableWakelock();
_requestBatteryOptimizationExemption();
if (context.read<OverlayProvider>().isOverlayActive) {
context.read<OverlayProvider>().pushCurrentData();
}
}
Future<void> _requestBatteryOptimizationExemption() async {
try {
// Platform-specific implementation would go here
} catch (e) {
Logger.warn('Battery optimization exemption not available: $e', 'Main');
}
}
String _getSpeedDisplayText(double displaySpeed) {
// TEMPORARILY disabled for indoor testing - show all speeds
// TODO: Re-enable: if (_currentGpsData.speed < 1.0 && (_currentGpsData.heading < 0.0 || _currentGpsData.heading >= 360.0)) return '--';
return displaySpeed.toStringAsFixed(1);
}
Widget _buildSpeedDisplay(String speedText, ColorTheme currentTheme, double fontSize) {
if (speedText == '--') { if (speedText == '--') {
return Text( return Text(
speedText, speedText,
style: TextStyle( style: TextStyle(
fontSize: fontSize, fontSize: fontSize,
fontWeight: FontWeight.w300, fontWeight: FontWeight.w300,
color: currentTheme.speedText, color: currentTheme.speedText.withValues(alpha: opacity),
fontFamily: 'DIN1451Alt', fontFamily: 'DIN1451Alt',
), ),
); );
@@ -199,7 +147,7 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
style: TextStyle( style: TextStyle(
fontSize: fontSize, fontSize: fontSize,
fontWeight: FontWeight.w300, fontWeight: FontWeight.w300,
color: currentTheme.speedText, color: currentTheme.speedText.withValues(alpha: opacity),
fontFamily: 'DIN1451Alt', fontFamily: 'DIN1451Alt',
), ),
), ),
@@ -208,7 +156,7 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
style: TextStyle( style: TextStyle(
fontSize: fontSize * 0.5, fontSize: fontSize * 0.5,
fontWeight: FontWeight.w300, fontWeight: FontWeight.w300,
color: currentTheme.speedTextSub, color: currentTheme.speedTextSub.withValues(alpha: opacity),
fontFamily: 'DIN1451Alt', fontFamily: 'DIN1451Alt',
), ),
), ),
@@ -268,17 +216,28 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
else ...[ else ...[
Flexible( Flexible(
flex: 7, flex: 7,
child: Selector<GpsDataManager, double>( child: Selector<GpsDataManager, ProcessedGpsData>(
selector: (_, gps) => gps.currentData.speed, selector: (_, gps) => gps.currentData,
builder: (context, speed, _) { builder: (context, data, _) {
final settings = context.read<SettingsProvider>(); final settings = context.read<SettingsProvider>();
final displaySpeed = settings.currentUnit.convert(speed); final displaySpeed = settings.currentUnit.convert(data.speed);
final speedText = _getSpeedDisplayText(displaySpeed);
// Calculate staleness
final age = DateTime.now().difference(data.timestamp);
final isDimmed = age >= OverlayConfig.STALENESS_DIM_THRESHOLD;
final showDashes = age >= OverlayConfig.STALENESS_DASH_THRESHOLD;
final staleOpacity = isDimmed ? OverlayConfig.STALENESS_DIM_OPACITY : 1.0;
final speedText = showDashes ? '--' : displaySpeed.toStringAsFixed(1);
return GestureDetector( return GestureDetector(
onTap: () => context.read<SettingsProvider>().cycleTheme(), onTap: () {
HapticFeedback.lightImpact();
context.read<SettingsProvider>().cycleTheme();
},
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: _buildSpeedDisplay(speedText, currentTheme, 200), child: _buildSpeedDisplay(speedText, currentTheme, 200, staleOpacity),
), ),
); );
}, },
@@ -296,7 +255,10 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
selector: (_, settings) => settings.currentUnit.label, selector: (_, settings) => settings.currentUnit.label,
builder: (context, unitLabel, _) { builder: (context, unitLabel, _) {
return GestureDetector( return GestureDetector(
onTap: () => context.read<SettingsProvider>().cycleUnit(), onTap: () {
HapticFeedback.lightImpact();
context.read<SettingsProvider>().cycleUnit();
},
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Text( child: Text(
@@ -338,22 +300,27 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
width: double.infinity, width: double.infinity,
color: Colors.transparent, color: Colors.transparent,
padding: const EdgeInsets.all(2), padding: const EdgeInsets.all(2),
child: Selector<GpsDataManager, (double heading, String displayHeading)>( child: Selector<GpsDataManager, ProcessedGpsData>(
selector: (_, gps) => (gps.currentData.heading, gps.currentData.displayHeading), selector: (_, gps) => gps.currentData,
builder: (context, data, _) { builder: (context, data, _) {
// Calculate staleness
final age = DateTime.now().difference(data.timestamp);
final isDimmed = age >= OverlayConfig.STALENESS_DIM_THRESHOLD;
final staleOpacity = isDimmed ? OverlayConfig.STALENESS_DIM_OPACITY : 1.0;
return Column( return Column(
mainAxisAlignment: MainAxisAlignment.center, mainAxisAlignment: MainAxisAlignment.center,
children: [ children: [
Flexible( Flexible(
flex: 3, flex: 3,
child: Transform.rotate( child: Transform.rotate(
angle: (data.$1 >= 0 && data.$1 < 360) ? (data.$1 * math.pi / 180.0) : 0, angle: (data.heading >= 0 && data.heading < 360) ? (data.heading * math.pi / 180.0) : 0,
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Icon( child: Icon(
Icons.navigation, Icons.navigation,
size: 80, size: 80,
color: currentTheme.headingText, color: currentTheme.headingText.withValues(alpha: staleOpacity),
), ),
), ),
), ),
@@ -363,10 +330,10 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Text( child: Text(
data.$2, data.displayHeading,
style: TextStyle( style: TextStyle(
fontSize: 50, fontSize: 50,
color: currentTheme.headingText, color: currentTheme.headingText.withValues(alpha: staleOpacity),
fontFamily: 'DIN1451Alt', fontFamily: 'DIN1451Alt',
), ),
), ),
@@ -415,17 +382,28 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
), ),
Flexible( Flexible(
flex: 20, flex: 20,
child: Selector<GpsDataManager, double>( child: Selector<GpsDataManager, ProcessedGpsData>(
selector: (_, gps) => gps.currentData.speed, selector: (_, gps) => gps.currentData,
builder: (context, speed, _) { builder: (context, data, _) {
final settings = context.read<SettingsProvider>(); final settings = context.read<SettingsProvider>();
final displaySpeed = settings.currentUnit.convert(speed); final displaySpeed = settings.currentUnit.convert(data.speed);
final speedText = _getSpeedDisplayText(displaySpeed);
// Calculate staleness
final age = DateTime.now().difference(data.timestamp);
final isDimmed = age >= OverlayConfig.STALENESS_DIM_THRESHOLD;
final showDashes = age >= OverlayConfig.STALENESS_DASH_THRESHOLD;
final staleOpacity = isDimmed ? OverlayConfig.STALENESS_DIM_OPACITY : 1.0;
final speedText = showDashes ? '--' : displaySpeed.toStringAsFixed(1);
return GestureDetector( return GestureDetector(
onTap: () => context.read<SettingsProvider>().cycleTheme(), onTap: () {
HapticFeedback.lightImpact();
context.read<SettingsProvider>().cycleTheme();
},
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: _buildSpeedDisplay(speedText, currentTheme, 160), child: _buildSpeedDisplay(speedText, currentTheme, 160, staleOpacity),
), ),
); );
}, },
@@ -437,7 +415,10 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
selector: (_, settings) => settings.currentUnit.label, selector: (_, settings) => settings.currentUnit.label,
builder: (context, unitLabel, _) { builder: (context, unitLabel, _) {
return GestureDetector( return GestureDetector(
onTap: () => context.read<SettingsProvider>().cycleUnit(), onTap: () {
HapticFeedback.lightImpact();
context.read<SettingsProvider>().cycleUnit();
},
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Text( child: Text(
@@ -477,22 +458,27 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
height: double.infinity, height: double.infinity,
color: Colors.transparent, color: Colors.transparent,
padding: const EdgeInsets.all(2), padding: const EdgeInsets.all(2),
child: Selector<GpsDataManager, (double heading, String displayHeading)>( child: Selector<GpsDataManager, ProcessedGpsData>(
selector: (_, gps) => (gps.currentData.heading, gps.currentData.displayHeading), selector: (_, gps) => gps.currentData,
builder: (context, data, _) { builder: (context, data, _) {
// Calculate staleness
final age = DateTime.now().difference(data.timestamp);
final isDimmed = age >= OverlayConfig.STALENESS_DIM_THRESHOLD;
final staleOpacity = isDimmed ? OverlayConfig.STALENESS_DIM_OPACITY : 1.0;
return Column( return Column(
mainAxisAlignment: MainAxisAlignment.center, mainAxisAlignment: MainAxisAlignment.center,
children: [ children: [
Flexible( Flexible(
flex: 3, flex: 3,
child: Transform.rotate( child: Transform.rotate(
angle: (data.$1 >= 0 && data.$1 < 360) ? (data.$1 * math.pi / 180.0) : 0, angle: (data.heading >= 0 && data.heading < 360) ? (data.heading * math.pi / 180.0) : 0,
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Icon( child: Icon(
Icons.navigation, Icons.navigation,
size: 60, size: 60,
color: currentTheme.headingText, color: currentTheme.headingText.withValues(alpha: staleOpacity),
), ),
), ),
), ),
@@ -502,10 +488,10 @@ class _SpeedometerScreenState extends State<SpeedometerScreen> with WidgetsBindi
child: FittedBox( child: FittedBox(
fit: BoxFit.contain, fit: BoxFit.contain,
child: Text( child: Text(
data.$2, data.displayHeading,
style: TextStyle( style: TextStyle(
fontSize: 30, fontSize: 30,
color: currentTheme.headingText, color: currentTheme.headingText.withValues(alpha: staleOpacity),
fontFamily: 'DIN1451Alt', fontFamily: 'DIN1451Alt',
), ),
), ),