Skip to content
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

Dead keys incorrectly handled by the terminal - issue is really in xterm.js #10990

Open
fperez opened this issue Sep 2, 2021 · 5 comments
Open
Labels
bug tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues

Comments

@fperez
Copy link
Contributor

fperez commented Sep 2, 2021

Description

This is really a bug in xterm.js - I'm opening this one so we can track its resolution over there and make the appropriate port (plus testing that any suggested fix actually works for us).

I've comented here on the xtermjs repo. That issue was recently closed (not sure why as I still see the problem on xtermjs.org), but hopefully the team will still respond. I'll update this if I end up opening a new issue there or they send me elsewhere.

Not much else to do here for now, til there's an upstream fix coming (or someone here knows what to do and wants to help the xterm team :)

Action: update our copy to 4.14 when released

As @Tyriar explains below, the problem is fixed in xterm.js and will be part of the 4.14 release. We can close this on our end then once we update to that version.

@fperez fperez added the bug label Sep 2, 2021
@Tyriar
Copy link

Tyriar commented Sep 2, 2021

FYI xtermjs.org is on the stable 4.13 so the fixes won't be there yet. This is fixed afaik with xtermjs/xterm.js#3430, xtermjs/xterm.js#3432 and xtermjs/xterm.js#3437

@fperez
Copy link
Contributor Author

fperez commented Sep 2, 2021

Ah, cool, thanks @Tyriar, I understand now.

Great to know this is now getting fixed upstream, we can update then the version on our end when 4.14 is out with these improvements.

I hope the comment about the PC-International layout is useful to your team - I've used that layout on Macs, Windows and Linux and it offers an easy way to get dead keys on regular US keyboards, so perhaps it can be of help. I saw some comments about folks having a hard time testing some of this behavior b/c they only had US keyboards, which is why I mentioned it.

Thanks for the quick response, and more importantly the great tool! We really benefit a lot from it in Jupyter, so your work is much appreciated.

@arivero
Copy link

arivero commented Sep 15, 2021

Surely related to this, the terminal shows a strange layout when I use a Spanish Keyboard

\1234567890[]
qwertyuiop[=
asdfghjkl`'/
\zxcvbnm,.-
|!@#$%^&*(){}
QWERTYUIOP{+
ASDFGHJKL~"?
|ZXCVBNM<>_

@pacha
Copy link

pacha commented Oct 9, 2021

Hi, it seems that xtermjs 4.14.1 is already available (https://github.com/xtermjs/xterm.js/releases/tag/4.14.1). @fperez do you know if there are plans to update the version of the library in JupyterLab?
Also, many thanks to xtermjs for fixing the issue! 🙂

@fperez
Copy link
Contributor Author

fperez commented Oct 9, 2021

I'm not sure if there are any blockers to include this in a future JLab release, but it would be great to see it go in... I'll try to do a bit of testing and will report back.

yuvipanda added a commit to yuvipanda/jupyterlab that referenced this issue Jun 18, 2022
Ref jupyterlab#10990

Seems to improve deadkeys behavior in Safari but not Firefox
fcollonval pushed a commit to yuvipanda/jupyterlab that referenced this issue Jun 27, 2022
Ref jupyterlab#10990

Seems to improve deadkeys behavior in Safari but not Firefox
fcollonval pushed a commit to yuvipanda/jupyterlab that referenced this issue Jun 30, 2022
Ref jupyterlab#10990

Seems to improve deadkeys behavior in Safari but not Firefox
fcollonval added a commit that referenced this issue Jun 30, 2022
* Bump xtermjs to latest

Ref #10990

Seems to improve deadkeys behavior in Safari but not Firefox

* Update Playwright snapshots

Co-authored-by: Frédéric Collonval <fcollonval@gmail.com>
@JasonWeill JasonWeill added the tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues label Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug tag:Keyboard Compatibility issues with keyboards, including locale-specific and layout-specific issues
Projects
None yet
Development

No branches or pull requests

5 participants