Skip to content

Document Android backspace issue #3079

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

Closed
1 task
jsjoeio opened this issue Apr 8, 2021 · 15 comments · Fixed by #3251
Closed
1 task

Document Android backspace issue #3079

jsjoeio opened this issue Apr 8, 2021 · 15 comments · Fixed by #3251
Assignees
Labels
docs Documentation related
Milestone

Comments

@jsjoeio
Copy link
Contributor

jsjoeio commented Apr 8, 2021

We need to indicate in the Termux docs there are known issues with the backspace key on Android (using on-screen keyboard).

See comment

Possibly:

  • add new docs/termux.md file and add there.
@jsjoeio jsjoeio added the docs Documentation related label Apr 8, 2021
@jsjoeio jsjoeio added this to the v3.9.4 milestone Apr 8, 2021
@jsjoeio jsjoeio changed the title Document Document Android backspace issue Apr 8, 2021
@ghost
Copy link

ghost commented Apr 8, 2021

Backspace works if you start backspacing "in the middle" of the word you wanna delete, but it crashes code-server(? Proof here: https://youtu.be/WOGQubsOjoo

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 12, 2021

Argh, well at least we know there is a workaround.

@leKamikaze1 do you know if the issue occurs with a separate bluetooth keyboard? I can try testing too. Wondering if this is specific to the on-screen keyboard.

@ghost
Copy link

ghost commented Apr 12, 2021

Argh, well at least we know there is a workaround.

@leKamikaze1 do you know if the issue occurs with a separate bluetooth keyboard? I can try testing too. Wondering if this is specific to the on-screen keyboard.

Sadly, I never had a Bluetooth keyboard In my life, so its pretty impossible for me to test that

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 12, 2021

No worries at all. I have one lying around somewhere so I can test!

@ghost
Copy link

ghost commented Apr 15, 2021

@jsjoeio Found something?

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 15, 2021

I finally got around to testing - it works with a bluetooth keyboard! No backspace issues 😄

Demo

screen-20210415-110610.mp4

@ghost
Copy link

ghost commented Apr 15, 2021

Well, that's great. So, the issue is on Android's on-screen keyboard? Not sure what would cause it 🤔

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 15, 2021

It has to do with the way xterm.js is handling the input for on-screen keyboards I'm guessing:

Pulled from comment

@fax1ty
Copy link

fax1ty commented Apr 18, 2021

Any news? Working solution?
Backspace working, but can't delete special characters

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 19, 2021

Unfortunately, no :/ this is upstream and out of our control to fix. Only working solution I know of is to use an external keyboard.

@nikey02
Copy link

nikey02 commented Apr 25, 2021

microsoft/vscode@20f3ece#diff-720a2dcc75d4242d03024e70d87943ebc58dcdbb8baf1d1ab8b78792248b621a

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 26, 2021

Thanks for linking that @nikey02 ! Looks like that was part of vscode 1.55.2, which we upgraded to in #3085 which means...hopefully this isn't an issue in our next release 🤞

@nikey02
Copy link

nikey02 commented Apr 27, 2021

@jsjoeio
glad to hear that but
i want to point out some issues i facing

  • in code-server settings UI dropdown menu is not working
  • also regarding gboard backspace issue which was in monaco-editor but after patch another problem is shown in monaco-editor@0.23.0 that is backspace not
    working at beginning of the line

may be this is specific to me, if anyone has this issue please comment or test it ( all tested/having this in termux/android )

@jsjoeio
Copy link
Contributor Author

jsjoeio commented Apr 27, 2021

To keep things separate and ensure this doesn't get lost, do you mind opening a separate bug report issue? I'd be happy to take a closer look!

@nikey02
Copy link

nikey02 commented Apr 27, 2021

sure I'll make issue with proper testing and confirmation

thanks @jsjoeio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants