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

Hitting Shift while converting confirms current choice #55

Closed
uKLEina opened this issue Jul 26, 2018 · 4 comments
Closed

Hitting Shift while converting confirms current choice #55

uKLEina opened this issue Jul 26, 2018 · 4 comments

Comments

@uKLEina
Copy link

uKLEina commented Jul 26, 2018

Environment

  • Debian Buster
  • ibus 1.5.18
  • ibus-skk 1.4.2
  • libskk 1.0.4

also reproduced with fcitx 4.2.9.6 and fcitx-skk 0.1.4

Steps to reproduce

  1. Start input with Shift.
  2. Hit Space to start converting
  3. Hit Shift

Expected Result

Nothing happens.

Actual Result

The current selection is confirmed.

Notes

I use xcape for enabling SandS feature, and it sends Shift + Space when you hit space key once. So with current behavior, I can only select 1st candidate of converting. Also, in ddskk, hitting Shift while converting does nothing, so current behavior does not match with ddskk.
This happens after I updated libskk from 1.0.3 to 1.0.4, so I think libskk 1.0.4 causes this problem.

Edit: I tested with libskk 1.0.3 and this issue doesn't happen, so I think it's confirmed that libskk 1.0.4 causes this behavior.

@ueno
Copy link
Owner

ueno commented Jul 26, 2018

I'm guessing you are using the latest Debian package of libskk, which accidentally included the patch not merged in upstream:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903159#20

The similar symptoms were already reported in the review:
#54 (comment)

@ueno
Copy link
Owner

ueno commented Jul 27, 2018

cc @hosiet

@hosiet
Copy link

hosiet commented Jul 27, 2018

Ack. Will revert the unmerged patch if I could confirm the bug.

@ueno
Copy link
Owner

ueno commented Jul 31, 2018

Closing, as the new package has been submitted:
https://metadata.ftp-master.debian.org/changelogs/main/libs/libskk/libskk_1.0.4-2_changelog

@ueno ueno closed this as completed Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants