From b60db83faadbd5784d8babb8df4b9664f352cde9 Mon Sep 17 00:00:00 2001 From: Bartlomiej Moskal Date: Tue, 17 Sep 2024 09:20:08 +0200 Subject: [PATCH] Android: Allow to open Keyboard without touch After changes in f4050cc5ea7490ba3b8b2bb0a174559d7e72a27e commit, the keyboard can be opened only when internal m_currentEditText is set (what is happening only on touch). Because of that, the keyboard cannot be opened just by setting focus on an item like it was before. To allow open the keyboard not only after touch, QtWindow needs to be informed about each focus change. Fixes: QTBUG-124360 Change-Id: Ic3ca4451f53df55bfb1f3e300078fd1916e77155 Reviewed-by: Assam Boudjelthia (cherry picked from commit 2971dbfe062de6b0be3ed9f251f9506bd45d5854) Reviewed-by: Qt Cherry-pick Bot --- src/android/jar/src/org/qtproject/qt/android/QtWindow.java | 7 +++++++ src/plugins/platforms/android/qandroidplatformscreen.cpp | 4 +++- src/plugins/platforms/android/qandroidplatformwindow.cpp | 5 +++++ src/plugins/platforms/android/qandroidplatformwindow.h | 1 + 4 files changed, 16 insertions(+), 1 deletion(-) diff --git a/src/android/jar/src/org/qtproject/qt/android/QtWindow.java b/src/android/jar/src/org/qtproject/qt/android/QtWindow.java index 88078a843d..5ceccb3bf5 100644 --- a/src/android/jar/src/org/qtproject/qt/android/QtWindow.java +++ b/src/android/jar/src/org/qtproject/qt/android/QtWindow.java @@ -236,4 +236,11 @@ class QtWindow extends QtLayout implements QtSurfaceInterface { if (m_parentWindow != null) m_parentWindow.addChildWindow(this); } + + @UsedFromNativeCode + void updateFocusedEditText() + { + if (m_editText != null && m_inputConnectionListener != null) + m_inputConnectionListener.onEditTextChanged(m_editText); + } } diff --git a/src/plugins/platforms/android/qandroidplatformscreen.cpp b/src/plugins/platforms/android/qandroidplatformscreen.cpp index 4d752a3cc3..cc9c435a32 100644 --- a/src/plugins/platforms/android/qandroidplatformscreen.cpp +++ b/src/plugins/platforms/android/qandroidplatformscreen.cpp @@ -299,8 +299,10 @@ void QAndroidPlatformScreen::topVisibleWindowChanged() QtAndroidMenu::setActiveTopLevelWindow(w); if (w && w->handle()) { QAndroidPlatformWindow *platformWindow = static_cast(w->handle()); - if (platformWindow) + if (platformWindow) { platformWindow->updateSystemUiVisibility(); + platformWindow->updateFocusedEditText(); + } } } diff --git a/src/plugins/platforms/android/qandroidplatformwindow.cpp b/src/plugins/platforms/android/qandroidplatformwindow.cpp index 1362b7d392..5559a89759 100644 --- a/src/plugins/platforms/android/qandroidplatformwindow.cpp +++ b/src/plugins/platforms/android/qandroidplatformwindow.cpp @@ -266,6 +266,11 @@ void QAndroidPlatformWindow::updateSystemUiVisibility() } } +void QAndroidPlatformWindow::updateFocusedEditText() +{ + m_nativeQtWindow.callMethod("updateFocusedEditText"); +} + bool QAndroidPlatformWindow::isExposed() const { return qApp->applicationState() > Qt::ApplicationHidden diff --git a/src/plugins/platforms/android/qandroidplatformwindow.h b/src/plugins/platforms/android/qandroidplatformwindow.h index ef4e799a75..c282adb9b0 100644 --- a/src/plugins/platforms/android/qandroidplatformwindow.h +++ b/src/plugins/platforms/android/qandroidplatformwindow.h @@ -57,6 +57,7 @@ public: void propagateSizeHints() override; void requestActivateWindow() override; void updateSystemUiVisibility(); + void updateFocusedEditText(); inline bool isRaster() const { return m_isRaster; } bool isExposed() const override; QtJniTypes::QtWindow nativeWindow() const { return m_nativeQtWindow; }