[green] dismiss native keyboards before expanding the console

Signed-off-by: Viwat Vchirawongkwin <viwat.v@chula.ac.th>

Author: yyods

app/android/app/src/main/kotlin/dev/pyble/pyble/MainActivity.kt

@@ -1,5 +1,28 @@
+// SPDX-License-Identifier: MIT
+// Part of PyBLE (https://pyble.dev) — see /LICENSE.
+
 package dev.pyble.pyble
 
+import android.view.inputmethod.InputMethodManager
 import io.flutter.embedding.android.FlutterActivity
+import io.flutter.embedding.engine.FlutterEngine
+import io.flutter.plugin.common.MethodChannel
 
-class MainActivity : FlutterActivity()
+class MainActivity : FlutterActivity() {
+    override fun configureFlutterEngine(flutterEngine: FlutterEngine) {
+        super.configureFlutterEngine(flutterEngine)
+        MethodChannel(flutterEngine.dartExecutor.binaryMessenger, "dev.pyble/keyboard")
+            .setMethodCallHandler { call, result ->
+                if (call.method == "dismissKeyboard") {
+                    val manager = getSystemService(InputMethodManager::class.java)
+                    window.decorView.windowToken?.let { token ->
+                        manager?.hideSoftInputFromWindow(token, 0)
+                    }
+                    // Acknowledge the request; Flutter observes actual view insets.
+                    result.success(null)
+                } else {
+                    result.notImplemented()
+                }
+            }
+    }
+}

app/ios/Runner/AppDelegate.swift

@@ -1,3 +1,6 @@
+// SPDX-License-Identifier: MIT
+// Part of PyBLE (https://pyble.dev) — see /LICENSE.
+
 import Flutter
 import UIKit
 
@@ -12,5 +15,19 @@ import UIKit
 
   func didInitializeImplicitFlutterEngine(_ engineBridge: FlutterImplicitEngineBridge) {
     GeneratedPluginRegistrant.register(with: engineBridge.pluginRegistry)
+    guard let registrar = engineBridge.pluginRegistry.registrar(forPlugin: "PybleKeyboardDismissal")
+    else { return }
+    let keyboard = FlutterMethodChannel(
+      name: "dev.pyble/keyboard", binaryMessenger: registrar.messenger())
+    keyboard.setMethodCallHandler { [weak registrar] call, result in
+      guard call.method == "dismissKeyboard" else {
+        result(FlutterMethodNotImplemented)
+        return
+      }
+      // End editing in this engine's window, including embedded WKWebView fields.
+      registrar?.viewController?.viewIfLoaded?.window?.endEditing(true)
+      // Acknowledge the request; Flutter observes actual view insets.
+      result(nil)
+    }
   }
 }

app/lib/app/keyboard_dismissal.dart

@@ -0,0 +1,20 @@
+// SPDX-License-Identifier: MIT
+// Part of PyBLE (https://pyble.dev) — see /LICENSE.
+
+import 'package:flutter/foundation.dart';
+import 'package:flutter/services.dart';
+
+const MethodChannel _keyboardChannel = MethodChannel('dev.pyble/keyboard');
+
+/// Requests dismissal of the current window's keyboard, including native
+/// WebView fields. Completion does not assert that the keyboard is hidden;
+/// layout continues to follow the actual view insets.
+Future<void> dismissSoftwareKeyboard() async {
+  if (!kIsWeb &&
+      (defaultTargetPlatform == TargetPlatform.android ||
+          defaultTargetPlatform == TargetPlatform.iOS)) {
+    await _keyboardChannel.invokeMethod<void>('dismissKeyboard');
+  } else {
+    await SystemChannels.textInput.invokeMethod<void>('TextInput.hide');
+  }
+}

app/lib/app/shell.dart

@@ -25,6 +25,7 @@ import 'package:pyble/localization/localization.dart';
 import 'package:pyble/pble/pble.dart';
 import 'package:pyble/theme/theme.dart';
 
+import 'keyboard_dismissal.dart';
 import 'pages/about_page.dart';
 import 'pages/blocks_page.dart';
 import 'pages/connect_page.dart';
@@ -335,12 +336,13 @@ class _LandscapeShell extends ConsumerWidget {
               collapsed:
                   keyboardVisible || (blocksFocused && !blocksConsoleExpanded),
               onToggle: keyboardVisible
-                  ? () {
+                  ? () async {
                       if (blocksFocused) {
                         ref.read(blocksConsoleExpandedProvider.notifier).state =
                             true;
                       }
                       FocusScope.of(context).unfocus();
+                      await dismissSoftwareKeyboard();
                     }
                   : blocksFocused
                   ? () {

app/test/unit/keyboard_dismissal_test.dart

@@ -0,0 +1,21 @@
+// SPDX-License-Identifier: MIT
+// Part of PyBLE (https://pyble.dev) — see /LICENSE.
+
+import 'package:flutter/foundation.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:pyble/app/keyboard_dismissal.dart';
+
+void main() {
+  testWidgets(
+    'a platform without the app native handler uses the Flutter hide request',
+    (WidgetTester tester) async {
+      tester.testTextInput.log.clear();
+      await dismissSoftwareKeyboard();
+      expect(
+        tester.testTextInput.log.map((call) => call.method),
+        contains('TextInput.hide'),
+      );
+    },
+    variant: TargetPlatformVariant(<TargetPlatform>{TargetPlatform.linux}),
+  );
+}

app/test/widget/app_shell_test.dart

@@ -21,8 +21,6 @@
 // toolbar action slots) without weakening any of it.
 // HAND-OFF provenance: lib/app → app-architect; lib/pble → app-protocol-engineer.
 
-import 'dart:typed_data';
-
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/rendering.dart';
@@ -95,15 +93,9 @@ Future<void> _focusLandscapeBlocks(WidgetTester tester) async {
 
 void main() {
   group('landscape software keyboard', () {
-    for (final TargetPlatform platform in <TargetPlatform>[
-      TargetPlatform.android,
-      TargetPlatform.iOS,
-    ]) {
-      testWidgets('Expand dismisses a native Blocks keyboard on $platform', (
-        WidgetTester tester,
-      ) async {
-        debugDefaultTargetPlatformOverride = platform;
-        addTearDown(() => debugDefaultTargetPlatformOverride = null);
+    testWidgets(
+      'Expand requests native Blocks keyboard dismissal',
+      (WidgetTester tester) async {
         const MethodChannel keyboard = MethodChannel('dev.pyble/keyboard');
         final List<MethodCall> requests = <MethodCall>[];
         tester.binding.defaultBinaryMessenger.setMockMethodCallHandler(
@@ -154,8 +146,12 @@ void main() {
         );
         expect(tester.element(find.byKey(_blocksWorkspaceHostKey)), same(host));
         expect(tester.takeException(), isNull);
-      });
-    }
+      },
+      variant: TargetPlatformVariant(<TargetPlatform>{
+        TargetPlatform.android,
+        TargetPlatform.iOS,
+      }),
+    );
 
     testWidgets('Expand preserves a Flutter editor draft and selection', (
       WidgetTester tester,