-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
If keyboard toggle is enable/disable, you have to press it twice after keyboard is auto closed #2111
Comments
How
Yes, that's confirmed to be a bug as you described in #2112.
As I said, if you disable keyboard and exit termux and come back later, termux will still consider it in a disabled state at startup. It will only enable it again if you press the toggle in extra keys/drawer/settings. You can check the
Why should it do that? Back button is to hide the keyboard, not disable it. That will just take people out of the app when they don't want to, like to close keyboard to increase size of terminal view or to scroll.
Then users will be forced to use extra key button instead of tapping on terminal view to open keyboard again. |
Yes, that's not the issue here.
I wondered if I should open one or two issues, but it didn't seem like the same issue to me. This issue is about the keyboard being hidden after returning to Termux and the toggle button disabling it instead of showing it. #2112 is that it doesn't always open the keyboard when it enables it.
None of the steps involves exiting Termux, so that's irrelevant and the issue here is about what happens when the keyboard is enabled, not when it is disabled. The point is that if the keyboard is open, I want it to still be open after returning to Termux. Currently, if you lock the screen or open a URL and come back to Termux the keyboard is hidden, even though it was enabled and shown before. When I think about it, there is an inconsistency here even if you don't use the toggle button. If you go to the home screen and back to Termux, the keyboard will open, regardless if it was open or closed before. However, if you lock the screen or open a URL and go back to Termux, the keyboard will be closed, regardless if it was open or closed before. I think these should behave the same way, and that the keyboard should always be in the same state it was previously when you return to Termux.
I thought it would be more useful if the keyboard is always open when enabled if the option is set to enable/disable. But okay, if the bug that it doesn't open when the toggle is pressed is fixed, it's fine, so you can disregard this part about the back button. |
They are different issues, so two issues is fine.
Yeah, this is confirmed to be a bug too. I have fixed this issue, as well as "press toggle twice to show keyboard" locally, will send a pull request soon. Do you use debug builds or F-Droid? If there are no testers available using debug builds, then might as well push to |
Great, thanks! I'm building Termux myself, so I can test it as long as the code is pushed somewhere. |
Welcome, I guess I'll send a pull request then. |
1. If `soft-keyboard-toggle-behaviour=enable/disable` was set, then pressing keyboard toggle wouldn't show the keyboard after switching back from another app if keyboard was previously disabled by user. 2. If switching back from another app, like when opening url with context menu "Select URL" long press and returning to termux with back button, then soft keyboard wouldn't automatically open like it does on app startup. Fixes termux#2111, Fixes termux#2112
1. If `soft-keyboard-toggle-behaviour=enable/disable` was set, then pressing keyboard toggle wouldn't show the keyboard after switching back from another app if keyboard was previously disabled by user. 2. If switching back from another app, like when opening url with context menu "Select URL" long press and returning to termux with back button, then soft keyboard wouldn't automatically open like it does on app startup. Also fixed issue where OnFocusChangeListener wasn't being set up if keyboard had to be hidden or disabled on startup. Fixes termux#2111, Fixes termux#2112
1. If `soft-keyboard-toggle-behaviour=enable/disable` was set, then pressing keyboard toggle wouldn't show the keyboard after switching back from another app if keyboard was previously disabled by user. 2. If switching back from another app, like when opening url with context menu "Select URL" long press and returning to termux with back button, then soft keyboard wouldn't automatically open like it does on app startup. Also fixed issue where OnFocusChangeListener wasn't being set up if keyboard had to be hidden or disabled on startup. Fixes termux#2111, Fixes termux#2112
1. If `soft-keyboard-toggle-behaviour=enable/disable` was set, then pressing keyboard toggle wouldn't show the keyboard after switching back from another app if keyboard was previously disabled by user. 2. If switching back from another app, like when opening url with context menu "Select URL" long press and returning to termux with back button, then soft keyboard wouldn't automatically open like it does on app startup. Also fixed issue where OnFocusChangeListener wasn't being set up if keyboard had to be hidden or disabled on startup. Fixes termux#2111, Fixes termux#2112
1. If `soft-keyboard-toggle-behaviour=enable/disable` was set, then pressing keyboard toggle wouldn't show the keyboard after switching back from another app if keyboard was previously disabled by user. 2. If switching back from another app, like when opening url with context menu "Select URL" long press and returning to termux with back button, then soft keyboard wouldn't automatically open like it does on app startup. Also fixed issue where OnFocusChangeListener wasn't being set up if keyboard had to be hidden or disabled on startup. Fixes termux#2111, Fixes termux#2112
Problem description
With the option introduced in e4e638b for #1098, I'm having some issues with the keyboard auto closing. When this happens, and I press the keyboard toggle, nothing happens until I press it a second time (because the keyboard is disabled on the first press even though it's closed). Personally, I don't think the keyboard should auto close when having this option set, it should always be shown when the keyboard enabled.
Steps to reproduce
soft-keyboard-toggle-behaviour = enable/disable
.Alternatively:
soft-keyboard-toggle-behaviour = enable/disable
.Expected behavior
The keyboard should still be open in Termux after these steps. Currently, it's closed which causes the issue mentioned initially about having to press the keyboard toggle twice.
Additional information
When the
back-key
option is default/back, the keyboard is closed on back. This causes the same issue that you have to press the keyboard toggle twice. I think when the keyboard is set to enable/disable, the back key should either take you straight out of the app without closing the keyboard (like when you press back after the keyboard is closed), or alternatively disable the keyboard.The text was updated successfully, but these errors were encountered: