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 require mouse cursor #26

Open
tlonic opened this issue Jul 7, 2019 · 4 comments
Open

Dead keys require mouse cursor #26

tlonic opened this issue Jul 7, 2019 · 4 comments

Comments

@tlonic
Copy link

tlonic commented Jul 7, 2019

Hi, I'm having an issue with dead keys requiring the mouse to be over the urxvt window, as described in this issue: stepb#17

I've tried the latest version you have in this repo, which supposedly has a fix for this issue, but it does not seem to make a difference. Dead keys work as expected without this extension.

I have no issues with Shift+PgUp/Dn, just with dead keys. It's also worth noting that this issue also exists in the tabbed extension (Shift+PgUp/Dn work, though)

distro: ArchLinux 5.1.15-arch1-1-ARCH
urxvt: version 9.22
ibus: 1.5.20

.Xresources:

URxvt.scrollstyle: plain
URxvt.perl-ext-common: tabbedex
URxvt.letterSpace: = -1
URxvt.inputMethod: ibus
URxvt.preeditType: OverTheSpot

@mina86
Copy link
Owner

mina86 commented Jul 7, 2019

This is a known issue and I had never had time to look into it. Honestly I’m mostly just waiting for it to be fixed upstream or someone else figuring it out.

I've tried the latest version you have in this repo, which supposedly has a fix for this issue, but it does not seem to make a difference.

There were two separate issues: PgUp/Dn not working and dead keys not working. Conversation on stepb#17 is confusing because at the time I believed those to be a single bug but in reality only the PgUp/Dn had been fixed.

@tlonic
Copy link
Author

tlonic commented Jul 7, 2019

I see. I'm willing to try to figure out this issue since it's become quite an annoyance to me -- do you have any ideas about where to start looking? Also, do you know whether the urxvt devs are aware of the issue?

I have a minimal setup where I'm not running ibus and the issue is still present, so that doesn't seem to be a crucial element, although I also see the issue with ibus IMEs, so it interacts with ibus in some way.

@mina86
Copy link
Owner

mina86 commented Jul 7, 2019

I’d start by looking at what refresh_check function does since that’s what fixed PgDn/PgUp issue. There’s also this old patch which might point you at place to look at in urxvt source code.

And I don’t recall whether this issue had been brought up upstream. Couldn’t find a thread on mailing list either.

bk added a commit to bk/urxvt-tabbedex that referenced this issue Nov 18, 2020
When the mouse cursor is outside the active urxvt window dead
keys stop working, at least when using ibus. This small fix stops
the tabbedex extension from doing anyting when dead keys are
pressed under these circumstances, which sidesteps the issue.
See also mina86#26.
mina86 pushed a commit that referenced this issue Nov 18, 2020
When the mouse pointer is outside of the terminal window, dead keys
stop working.  When ibus is used, the prompt pops up but once next
key is pressed, no character is inserted to the terminal.  This fixes
that issue.

Unfortunately, the fix only works when ibus is used.  If urxvt is
configured not to utilise it, dead keys remain broken when the mouse
pointer is outside of the terminal window and result.

See: #26
Fixes: #39

[[email protected]: moved check to key_press handler and reworded
 comment and commit message]
mina86 pushed a commit that referenced this issue Nov 19, 2020
When the mouse pointer is outside of the terminal window, dead keys
stop working.  When ibus is used, the prompt pops up but once next
key is pressed, no character is inserted to the terminal.  This fixes
that issue.

Unfortunately, the fix only works when ibus is used.  If urxvt is
configured not to utilise it, dead keys remain broken when the mouse
pointer is outside of the terminal window and result.

See: #26
Fixes: #39

[[email protected]: moved check to key_press handler and reworded
 comment and commit message]
@mina86
Copy link
Owner

mina86 commented Nov 19, 2020

For anyone following along, this is now partially fixed. (By fixed I mean fix is in master branch though there hasn’t been a new release yet) Specifically, dead keys will now work urxvt uses ibus.

Dead keys still won’t work without it so keeping this issue open.

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

Successfully merging a pull request may close this issue.

2 participants