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

%selkey cannot specify characters that needs shift key to access #3

Open
sltam opened this issue May 24, 2014 · 7 comments
Open

%selkey cannot specify characters that needs shift key to access #3

sltam opened this issue May 24, 2014 · 7 comments
Assignees

Comments

@sltam
Copy link

sltam commented May 24, 2014

I have these config in my .cin file:

%gen_inp
...
%selkey !@#$%^&*()
...
%keyname begin
...

When I put this cin file into OpenVanilla 1.0.x, and type anything that has disambiguation, it shows !, @, etc. on top of the choices, but when I hit shift-1, shift-2, etc. OpenVanilla unexpectedly cancels the current composition and emits the ! character! This renders my input method completely unusable!

This does not happen to OpenVanilla 0.8.x.

Before:
2014-05-24 21 39 41

After hitting "!":
2014-05-24 21 40 05

@sltam
Copy link
Author

sltam commented May 24, 2014

Note that this issue is almost non-existent for cin files shipped with OpenVanilla, because they almost avoided %selkey of characters on the upper row of an en-us keyboard.

$ grep selkey openvanilla-1.0/DataTables/*.cin | grep -v 123456789
openvanilla-1.0/DataTables/array26.cin:%selkey 1234
openvanilla-1.0/DataTables/array40.cin:%selkey '[]-="{}_
openvanilla-1.0/DataTables/dayi3-patched.cin:%selkey '[]-\67890
openvanilla-1.0/DataTables/dayi3.cin:%selkey '[]-\6789
openvanilla-1.0/DataTables/lookup-poj.cin:%selkey asdfghjkl;
openvanilla-1.0/DataTables/poj-holo.cin:%selkey asdfghjkl;

Except 行列40, so here is a test case for it. Type: 8 =. You see this:

2014-05-24 21 55 16

Hit ". You expect to get but you get " instead.

@ghost
Copy link

ghost commented Jul 21, 2016

Have anyone been able to fix this?

Or does anyone know where to edit in the code to change this?

@sltam
Copy link
Author

sltam commented Oct 8, 2016

@etang77 My guess is OVInputMethodController::handleOVKey expecting OVOneDimensionalCandidatePanelImpl::handleKey to return CandidateSelected but it doesn't. It could be as simple as OVInputMethodController::handleEvent sending the wrong OVKey to handleOVKey as this function has plenty of key conversion logic.

Since I am not affiliated to this project, I don't know how to start helping. I can't even get my self-compiled library to run locally yet.

@ghost
Copy link

ghost commented Jan 19, 2017

I've seen that assocation phrases has in its script, allow to use Shift + Symbols as select key, and display as an "Up arrow 1". I'm sure it can work for the normal select key too, anyone can teach me how to edit?

@lukhnos
Copy link
Collaborator

lukhnos commented Jan 19, 2017

@sltam @etang77 If you use Xcode, could you try building the target "OpenVanillaInstaller" in the latest master (open OpenVanilla.xcodeproj in the repo root), and use the installer to install a dev version (1.3.2-dev) on your machine? The commit should fix this long-standing issue. Thanks!

@ghost
Copy link

ghost commented Jan 19, 2017

Thank you, so so so so much! Everything fixed (I think)!

@ghost
Copy link

ghost commented Feb 6, 2017

Being greedy here. Is it possible for the interface to show "Shift1"、"Shift2" like in Associated Phrase?

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

No branches or pull requests

2 participants