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

Allow page size to be set to 0 #206

Merged
merged 1 commit into from
Dec 21, 2024
Merged

Allow page size to be set to 0 #206

merged 1 commit into from
Dec 21, 2024

Conversation

wengxt
Copy link
Member

@wengxt wengxt commented Dec 17, 2024

Also, when page size is 0, change some behavior implicitly to allow simulate a
compose table like experience.

Fix #205

Also, when page size is 0, change some behavior implicitly to allow simulate a
compose table like experience.

Fix #205
@iamchloya
Copy link

(My settings):
изображение

When page size is 1 (I move cursor by pressing left arrow. After pressing, the letter has written in text.):
изображение

When page size is 0 (i do same actions, but the letter was not writing. If i switch to another program, the letter disappears):
изображение
(attention to underlining).

@iamchloya
Copy link

i has used it with in /home/username/.local/share/fcitx5/
myinputmethod.tar.gz

If I need to enter a word that ends in a letter (which used in combination), then I need to press return/enter twice. I disabled the confirmation on return/enter, but I still need to press twice. (Example: most, post)

Also, "ctrl+c" is blocked in the terminal if I do not press enter/arrow left or right/ to "complete" the letter writting

Should I create a separate issue?

@wengxt
Copy link
Member Author

wengxt commented Dec 19, 2024

@kuroya2mouse thank you for testing! Actually I did fixed that but I forgot to push it. Would you mind give it a try again?

@wengxt
Copy link
Member Author

wengxt commented Dec 19, 2024

@kuroya2mouse Also, I suggest you to set PreeditCursorPositionAtBeginning to false. That should make it feel more natural.

@iamchloya
Copy link

iamchloya commented Dec 19, 2024

IT'S REALLY WORKS. THANKS THANKS THANKS THANKS THANKS

If i notice any bugs, i'll let you know

@wengxt wengxt merged commit ed1a886 into master Dec 21, 2024
4 checks passed
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 this pull request may close these issues.

Add option to disable candidate list for table engine
2 participants