Fix: View pop with confirmation (#5859)

* Improve view pop confirmation and update routing example

Enhanced pop confirmation logic in ViewControl by introducing _allowPop flag and handling dialogs listener removal. Updated Python routing example to use ft.run(main) and added a new pop_view_confirm.py example demonstrating custom pop confirmation dialogs.

* Refactor routing_navigation examples and add integration tests

Refactored Python routing_navigation example apps to use 'if __name__ == "__main__"' for entry points and unified Flet imports. Updated GitHub Actions workflows to use kuhnroyal/flutter-fvm-config-action/setup@v3 for Flutter setup and improved test matrix. Added integration tests for routing_navigation examples to verify navigation and view logic.

* Add bounds check to Pagelet and new integration tests

Introduces a bounds check in Pagelet to display an error when height is unbounded and not explicitly set. Adds new integration tests and golden images for Pagelet sizing scenarios, including explicitly sized and auto-sized cases.

Close #5562

* Refactor imports for consistency and clarity

Reorganized and standardized import statements across multiple files, replacing relative and package imports with a consistent style. This improves code readability, maintainability, and reduces potential import conflicts.

* Add drawer navigation example and tests

Introduces a new drawer navigation example app and corresponding integration test. Refactors BasePage to use top/root view for drawer and related properties/methods, improving multi-view navigation support.

* Update test scope and remove golden image

Changed the pytest mark for test_cupertino_adaptive to use function scope instead of module scope. Deleted the macOS golden image for pagelet basic, possibly due to test update or asset cleanup.

* Update macOS pagelet basic golden image

Replaces the basic.png golden image for macOS pagelet integration tests to reflect updated UI output.

* Remove unused logging and update test icon buttons

Removed unused logging setup from drawer_navigation.py. Updated test_pagelet.py to remove explicit icon_color assignments for IconButton controls in integration tests. Updated golden images to reflect these changes.

Author: FeodorFitsner

.github/workflows/ci.yml

@@ -235,27 +235,11 @@ jobs:
       - name: Setup uv
         uses: astral-sh/setup-uv@v6
 
-      - name: Configure Flutter version
-        uses: kuhnroyal/flutter-fvm-config-action/config@v3
-        id: fvm-config-action
-        with:
-          path: '.fvmrc'
-
       - name: Setup Flutter
-        id: flutter-action
-        uses: subosito/flutter-action@v2
-        with:
-          flutter-version: ${{ steps.fvm-config-action.outputs.FLUTTER_VERSION }}
-          channel: ${{ steps.fvm-config-action.outputs.FLUTTER_CHANNEL }}
-          cache: false
-
-      # TODO: Remove when https://github.com/actions/runner-images/issues/13341 is fixed
-      - name: Cache Flutter
-        id: cache-flutter
-        uses: actions/cache@v4
+        uses: kuhnroyal/flutter-fvm-config-action/setup@v3
         with:
-          path: ${{ steps.flutter-action.outputs.CACHE-PATH }}
-          key: ${{ steps.flutter-action.outputs.CACHE-KEY }}
+          path: '.fvmrc'
+          cache: true
 
       - name: Build Flutter macOS client
         shell: bash

.github/workflows/macos-integration-tests.yml

@@ -36,7 +36,18 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        suite: [apps, examples, controls/core, controls/material, controls/cupertino, controls/theme, controls/types]
+        suite:
+        - apps
+        - examples/apps
+        - examples/core
+        - examples/cupertino
+        - examples/material
+        - controls/core
+        - controls/cupertino
+        - controls/material
+        - controls/theme
+        - controls/types
+
     name: ${{ matrix.suite }} Integration Tests
     steps:
       - name: Checkout repository
@@ -45,27 +56,11 @@ jobs:
       - name: Setup uv
         uses: astral-sh/setup-uv@v6
 
-      - name: Configure Flutter version
-        uses: kuhnroyal/flutter-fvm-config-action/config@v3
-        id: fvm-config-action
-        with:
-          path: '.fvmrc'
-
       - name: Setup Flutter
-        id: flutter-action
-        uses: subosito/flutter-action@v2
+        uses: kuhnroyal/flutter-fvm-config-action/setup@v3
         with:
-          flutter-version: ${{ steps.fvm-config-action.outputs.FLUTTER_VERSION }}
-          channel: ${{ steps.fvm-config-action.outputs.FLUTTER_CHANNEL }}
-          cache: false
-
-      # TODO: Remove when https://github.com/actions/runner-images/issues/13341 is fixed
-      - name: Cache Flutter
-        id: cache-flutter
-        uses: actions/cache@v4
-        with:
-          path: ${{ steps.flutter-action.outputs.CACHE-PATH }}
-          key: ${{ steps.flutter-action.outputs.CACHE-KEY }}
+          path: '.fvmrc'
+          cache: true
 
       - name: Show tool versions
         run: |

packages/flet/lib/src/controls/app_bar.dart

@@ -1,10 +1,10 @@
-import 'package:flet/src/utils/edge_insets.dart';
 import 'package:flutter/material.dart';
 
 import '../extensions/control.dart';
 import '../models/control.dart';
 import '../utils/borders.dart';
 import '../utils/colors.dart';
+import '../utils/edge_insets.dart';
 import '../utils/misc.dart';
 import '../utils/numbers.dart';
 import '../utils/text.dart';

packages/flet/lib/src/controls/auto_complete.dart

@@ -1,8 +1,8 @@
-import 'package:flet/src/utils/numbers.dart';
 import 'package:flutter/material.dart';
 
 import '../models/control.dart';
 import '../utils/auto_complete.dart';
+import '../utils/numbers.dart';
 import 'base_controls.dart';
 
 class AutoCompleteControl extends StatefulWidget {

packages/flet/lib/src/controls/bottom_app_bar.dart

@@ -1,6 +1,14 @@
-import 'package:flet/flet.dart';
 import 'package:flutter/material.dart';
 
+import '../extensions/control.dart';
+import '../models/control.dart';
+import '../utils/borders.dart';
+import '../utils/colors.dart';
+import '../utils/edge_insets.dart';
+import '../utils/misc.dart';
+import '../utils/numbers.dart';
+import 'base_controls.dart';
+
 class BottomAppBarControl extends StatelessWidget {
   final Control control;
 

packages/flet/lib/src/controls/button.dart

@@ -1,6 +1,16 @@
-import 'package:flet/flet.dart';
 import 'package:flutter/material.dart';
 
+import '../extensions/control.dart';
+import '../models/control.dart';
+import '../utils/buttons.dart';
+import '../utils/colors.dart';
+import '../utils/launch_url.dart';
+import '../utils/misc.dart';
+import '../utils/numbers.dart';
+import '../widgets/error.dart';
+import '../widgets/flet_store_mixin.dart';
+import 'base_controls.dart';
+
 class ButtonControl extends StatefulWidget {
   final Control control;
 

packages/flet/lib/src/controls/canvas.dart

@@ -1,21 +1,21 @@
 import 'dart:async';
 import 'dart:io';
-import 'dart:typed_data';
 import 'dart:ui' as ui;
 
-import 'package:flet/src/extensions/control.dart';
-import 'package:flet/src/utils/alignment.dart';
-import 'package:flet/src/utils/borders.dart';
-import 'package:flet/src/utils/colors.dart';
-import 'package:flet/src/utils/drawing.dart';
-import 'package:flet/src/utils/numbers.dart';
 import 'package:flutter/material.dart';
+import 'package:flutter/services.dart';
 import 'package:http/http.dart' as http;
 
+import '../extensions/control.dart';
 import '../models/control.dart';
+import '../utils/alignment.dart';
+import '../utils/borders.dart';
+import '../utils/colors.dart';
 import '../utils/dash_path.dart';
+import '../utils/drawing.dart';
 import '../utils/hashing.dart';
 import '../utils/images.dart';
+import '../utils/numbers.dart';
 import '../utils/text.dart';
 import '../utils/transforms.dart';
 import 'base_controls.dart';

packages/flet/lib/src/controls/circle_avatar.dart

@@ -1,6 +1,12 @@
-import 'package:flet/flet.dart';
 import 'package:flutter/material.dart';
 
+import '../extensions/control.dart';
+import '../models/control.dart';
+import '../utils/colors.dart';
+import '../utils/images.dart';
+import '../utils/numbers.dart';
+import 'base_controls.dart';
+
 class CircleAvatarControl extends StatelessWidget {
   final Control control;
 

packages/flet/lib/src/controls/container.dart

@@ -1,6 +1,22 @@
-import 'package:flet/flet.dart';
 import 'package:flutter/material.dart';
 
+import '../extensions/control.dart';
+import '../models/control.dart';
+import '../utils/alignment.dart';
+import '../utils/animations.dart';
+import '../utils/borders.dart';
+import '../utils/box.dart';
+import '../utils/colors.dart';
+import '../utils/edge_insets.dart';
+import '../utils/events.dart';
+import '../utils/gradient.dart';
+import '../utils/images.dart';
+import '../utils/launch_url.dart';
+import '../utils/misc.dart';
+import '../utils/numbers.dart';
+import '../widgets/flet_store_mixin.dart';
+import 'base_controls.dart';
+
 class ContainerControl extends StatelessWidget with FletStoreMixin {
   final Control control;
 

packages/flet/lib/src/controls/context_menu.dart

@@ -1,12 +1,12 @@
 import 'dart:async';
 
 import 'package:collection/collection.dart';
-import 'package:flet/src/utils/numbers.dart';
 import 'package:flutter/gestures.dart';
 import 'package:flutter/material.dart';
 
 import '../extensions/control.dart';
 import '../models/control.dart';
+import '../utils/numbers.dart';
 import '../utils/popup_menu.dart';
 import '../utils/transforms.dart';
 import '../widgets/error.dart';