Clean up debug logging

This commit is contained in:
stopflock
2025-12-07 15:09:31 -06:00
parent 405ec220d0
commit 7fb467872a
3 changed files with 4 additions and 27 deletions
@@ -37,17 +37,10 @@ class SheetCoordinator {
/// Get the currently active sheet height for map positioning
double get activeSheetHeight {
final height = _addSheetHeight > 0 ? _addSheetHeight :
_editSheetHeight > 0 ? _editSheetHeight :
_navigationSheetHeight > 0 ? _navigationSheetHeight :
_tagSheetHeight;
// Add debug logging to help troubleshoot sheet height coordination
if (height > 0) {
debugPrint('[SheetCoordinator] Active sheet height: $height (add: $_addSheetHeight, edit: $_editSheetHeight, nav: $_navigationSheetHeight, tag: $_tagSheetHeight)');
}
return height;
if (_addSheetHeight > 0) return _addSheetHeight;
if (_editSheetHeight > 0) return _editSheetHeight;
if (_navigationSheetHeight > 0) return _navigationSheetHeight;
return _tagSheetHeight;
}
/// Update sheet state tracking
@@ -107,7 +100,6 @@ class SheetCoordinator {
bottom: MediaQuery.of(context).padding.bottom, // Only safe area, no keyboard
),
child: MeasuredSheet(
debugLabel: 'AddNode',
onHeightChanged: (height) {
_addSheetHeight = height + MediaQuery.of(context).padding.bottom;
onStateChanged();
@@ -171,7 +163,6 @@ class SheetCoordinator {
bottom: MediaQuery.of(context).viewInsets.bottom + MediaQuery.of(context).padding.bottom,
),
child: MeasuredSheet(
debugLabel: 'EditNode',
onHeightChanged: (height) {
final fullHeight = height + MediaQuery.of(context).viewInsets.bottom + MediaQuery.of(context).padding.bottom;
_editSheetHeight = fullHeight;
@@ -211,7 +202,6 @@ class SheetCoordinator {
bottom: MediaQuery.of(context).viewInsets.bottom + MediaQuery.of(context).padding.bottom,
),
child: MeasuredSheet(
debugLabel: 'Navigation',
onHeightChanged: (height) {
final fullHeight = height + MediaQuery.of(context).viewInsets.bottom + MediaQuery.of(context).padding.bottom;
_navigationSheetHeight = fullHeight;
-2
View File
@@ -301,7 +301,6 @@ class _HomeScreenState extends State<HomeScreen> with TickerProviderStateMixin {
bottom: MediaQuery.of(context).padding.bottom, // Only safe area, no keyboard
),
child: MeasuredSheet(
debugLabel: 'NodeTag',
onHeightChanged: (height) {
_sheetCoordinator.updateTagSheetHeight(
height + MediaQuery.of(context).padding.bottom,
@@ -359,7 +358,6 @@ class _HomeScreenState extends State<HomeScreen> with TickerProviderStateMixin {
bottom: MediaQuery.of(context).padding.bottom, // Only safe area, no keyboard
),
child: MeasuredSheet(
debugLabel: 'SuspectedLocation',
onHeightChanged: (height) {
_sheetCoordinator.updateTagSheetHeight(
height + MediaQuery.of(context).padding.bottom,
-11
View File
@@ -4,13 +4,11 @@ import 'package:flutter/material.dart';
class MeasuredSheet extends StatefulWidget {
final Widget child;
final ValueChanged<double> onHeightChanged;
final String? debugLabel; // Add debug label for troubleshooting
const MeasuredSheet({
super.key,
required this.child,
required this.onHeightChanged,
this.debugLabel,
});
@override
@@ -34,17 +32,8 @@ class _MeasuredSheetState extends State<MeasuredSheet> {
final height = renderBox.size.height;
if (height != _lastHeight) {
_lastHeight = height;
// Add debug logging to help troubleshoot height measurement issues
if (widget.debugLabel != null) {
debugPrint('[MeasuredSheet-${widget.debugLabel}] Height changed: $_lastHeight -> $height');
}
widget.onHeightChanged(height);
}
} else {
// Add debug logging for measurement failures
if (widget.debugLabel != null) {
debugPrint('[MeasuredSheet-${widget.debugLabel}] Failed to measure height: renderBox is null');
}
}
}