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

Fix composition text color #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

MuTsunTsai
Copy link

這個 PR 是為了修正 EasyIME/PIME#698 ,把兩個參數修改之後,用編譯出來的 PIMETextService.dll 取代掉原本的檔案測試執行真的可以正確輸入:

image

或者在 VS Code 當中輸入時也會跟當前的程式碼顏色一致:

image

@MuTsunTsai
Copy link
Author

等等,這樣改的話變成小畫家會有問題了。可能要設法比較兩種顏色哪一個跟背景的反差較大然後再採用才行。

@MuTsunTsai
Copy link
Author

嗯,不行,更糟糕的是在淺色模式當中這個 PR 的寫法根本反而會導致錯誤的結果。看起來還要再多研究才行了。

@MuTsunTsai MuTsunTsai closed this Nov 17, 2022
@MuTsunTsai
Copy link
Author

MuTsunTsai commented Nov 17, 2022

啊,原來解答就是根本把設定顏色那幾列程式碼去掉、讓系統自己去安排顏色就好了。至少這樣一來在 Win11 上面測試無論是深色淺色主題在記事本、小畫家或其它已知的應用程式中都是正確的。

不太確定當初寫上那幾列是否在舊的系統中是必要的?可能要看看有沒有辦法在舊系統上測試了。

@MuTsunTsai MuTsunTsai reopened this Nov 17, 2022
@darkj2k
Copy link

darkj2k commented Jan 30, 2023

這個問題在1.3.0-stable還是存在。

@fxliang
Copy link

fxliang commented Jul 13, 2023

应该用TF_CT_NONE,由应用自行决定颜色

@MuTsunTsai
Copy link
Author

应该用TF_CT_NONE,由应用自行决定颜色

去掉的的話不也是一樣地由應用程式自行決定顏色?

@fxliang
Copy link

fxliang commented Jul 14, 2023

应该用TF_CT_NONE,由应用自行决定颜色

去掉的的話不也是一樣地由應用程式自行決定顏色?

你这个pr上的是
COLOR_WINDOW,和COLOR_WINDOWTEXT呀……这个是指定了颜色了

@MuTsunTsai
Copy link
Author

@fxliang 你應該是看到第一個 commit 去了,我第二個是改成直接拿掉

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.

3 participants