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

Refactor for Candidats Windows and more #1567

Merged
merged 14 commits into from
Jan 27, 2025

Conversation

WhiredPlanck
Copy link
Collaborator

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes # N/A

Feature

Describe features of this pull request

See the commit history for detail.

Code of conduct

Code style

Build pass

  • make debug

Manually test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub Action CI pass
  4. At least one contributor review and approve
  5. Merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login and download artifact at https://github.com/osfans/trime/actions

Additional Info

…rectly

The spans for spanned string can influence the line break strategy in TextView, therefore, CandidateItemSpan is implemented to replace the original CandidateSpan to fix the issue.
@WhiredPlanck
Copy link
Collaborator Author

@Bambooin @goofyz All done. This PR is finally ready to review.

@WhiredPlanck
Copy link
Collaborator Author

Another small bug fix. No any more commit for this PR now.

@WhiredPlanck
Copy link
Collaborator Author

Preview

Screen_recording_20250126_215557_2.webm

@goofyz
Copy link
Collaborator

goofyz commented Jan 27, 2025

我的會覆蓋鍵盤,是因為方案設定的問題嗎?

image

@WhiredPlanck
Copy link
Collaborator Author

我的會覆蓋鍵盤,是因為方案設定的問題嗎?

image

目前窗口的定位还存在一些小问题,但我打算另开一个 PR 把其他相关的小修复给带上。这个 PR 的提交记录有点长了。

@goofyz
Copy link
Collaborator

goofyz commented Jan 27, 2025

Bug: 候選視窗若曾使用「始終顯示」,那麼就算在選項中轉去「系統預設」或「停用」,候選視窗還是會繼續顯示。要強制停止 trime 才能回復。

@WhiredPlanck
Copy link
Collaborator Author

Bug: 候選視窗若曾使用「始終顯示」,那麼就算在選項中轉去「系統預設」或「停用」,候選視窗還是會繼續顯示。要強制停止 trime 才能回復。

这个对于其他设置来说也是如此,是一个普遍 bug。这时由于对设置项的变化监听做得不完整导致的,同样我将在最近几个 PR 内修复。

@WhiredPlanck WhiredPlanck merged commit 6f4448c into osfans:develop Jan 27, 2025
3 checks passed
@WhiredPlanck WhiredPlanck deleted the refactor-20250122 branch January 27, 2025 15:08
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.

2 participants