sander/maps
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: skip redundant mapController.move() on gesture-driven pans #1

Merged
sander merged 1 commit from claude/relaxed-wright into main 2026-04-04 14:06:34 +00:00
Conversation 0 Commits 1 Files changed 1 +10 -1
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
mobile/lib/features/map/presentation/screens/map_screen.dart
View file

@ -42,9 +42,18 @@ class _MapScreenState extends ConsumerState<MapScreen> {
final styleAsync = ref.watch(mapStyleProvider(apiClient.baseUrl));
// Listen for zoom/center changes from the provider and move the map.
// Skip the move if the map controller is already at the target position
// (meaning the state change came from a user gesture via onPositionChanged).
// Without this guard, every gesture-driven pan calls _mapController.move()
// with the position the map is already at, which triggers a tile
// recalculation that cancels in-flight tile loads and produces a
// partially-rendered (diagonal) map.
ref.listen<MapState>(mapProvider, (previous, next) {
if (previous?.center != next.center || previous?.zoom != next.zoom) {
_mapController.move(next.center, next.zoom);
final camera = _mapController.camera;
if (camera.center != next.center || camera.zoom != next.zoom) {
_mapController.move(next.center, next.zoom);
}
}
});