Skip to content

Dev#277

Merged
nhktmdzhg merged 20 commits into
mainfrom
dev
May 14, 2026
Merged

Dev#277
nhktmdzhg merged 20 commits into
mainfrom
dev

Conversation

@nhktmdzhg
Copy link
Copy Markdown
Collaborator

No description provided.

hthienloc and others added 19 commits May 2, 2026 05:00
Add DoubleHyphenToEmDash option following the same pattern as
DoubleSpaceToPeriod. When enabled and user types '--', it is
replaced with em-dash (—, U+2014).
- Add DoubleHyphenToEmDash to Python settings UI
- Support FcitxKey_KP_Subtract (numpad minus) for em-dash
- Check no modifiers pressed when triggering replacement
Add option to convert double hyphen to em-dash
---------

Co-authored-by: Nguyen Hoang Ky <nhktmdzhg@gmail.com>
- When at last item and press Down, wrap to first item of current page
- When at first item and press Up, wrap to last item of current page
- This provides consistent behavior across all pages without jumping between pages
---------

Co-authored-by: Nguyen Hoang Ky <nhktmdzhg@gmail.com>
Signed-off-by: Zebra2711 <zebra2711t@gmail.com>
fix: don’t update preedit in fake BS mode
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates fcitx5-lotus to version 3.1.0, introducing bracket transformation and double hyphen to em-dash conversion. Key changes include a refactored NixOS module with multi-user support and validation, UI label simplifications, and the removal of the deletion monitor thread and its associated synchronization logic. Feedback focuses on the updated navigation logic for candidate lists (Tab, Down, and Up keys), noting that wrapping within the current page is non-standard and suggesting a wrap-around for the entire list to maintain consistency with standard behavior.

Comment thread src/lotus-state.cpp
Comment thread src/lotus-state.cpp
@nhktmdzhg nhktmdzhg merged commit 63239d1 into main May 14, 2026
39 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Kanban May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants