Skip to content

fix(android): modify automatically closing the keyboard after typed text #793

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

welkang
Copy link

@welkang welkang commented May 30, 2025

主要解决安卓输入文字后会自动关闭键盘,有些页面逻辑识别到键盘关闭,会将页面的下一步操作按钮隐藏,导致流程无法继续下去。

该 PR 将关闭键盘键盘行为移除,后续可以考虑是否改成可配置化。

@CLAassistant
Copy link

CLAassistant commented May 30, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

netlify bot commented May 30, 2025

Deploy Preview for midscene ready!

Name Link
🔨 Latest commit 65f9517
🔍 Latest deploy log https://app.netlify.com/projects/midscene/deploys/6839c5599414690008a31ccf
😎 Deploy Preview https://deploy-preview-793--midscene.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@welkang welkang force-pushed the fix/android-hide-keyboard branch from e241f92 to 65f9517 Compare May 30, 2025 14:48
* Whether to hide the keyboard after typing is completed
* @default true
*/
hide?: boolean;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about rename as autoDismissKeyboard ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This naming is indeed more semantically clear. I'll make the change.

@yuyutaotao yuyutaotao requested a review from quanru June 3, 2025 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants