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

color_scheme_dark 配置不能立即生效 #1066

Closed
3 tasks done
RiwiHow opened this issue Dec 26, 2023 · 6 comments · Fixed by #1055
Closed
3 tasks done

color_scheme_dark 配置不能立即生效 #1066

RiwiHow opened this issue Dec 26, 2023 · 6 comments · Fixed by #1055

Comments

@RiwiHow
Copy link

RiwiHow commented Dec 26, 2023

上报前请检查

  • 我遇到的问题没有其他人在 issue 里提到过
  • 我的小狼毫版本于 rime/weasel 下载
  • 我在使用小狼毫的最新发布版本,或最新发布版本后的 CI 构建

操作系统信息

  • OS 详细版本: [ Windows 11 23H2 22631.2861]
  • 小狼毫版本: [ 最新的 CI 版本(1e78320) ]

描述遇到的问题
color_scheme_dark 配置不能立即生效

复现步骤
重现问题的步骤,如:

  1. 现在在 Windows 设置里面是「浅色」,输入法是 color_scheme 的配色
  2. 改变设置为「深色」,输入法依旧是 color_scheme 的配色,必须手动「重启算法服务」后,输入法才能变成 color_scheme_dark 的配色
  3. 从「深色」切换成「浅色」,也需要手动「重启算法服务」

预期行为
Windows 颜色改变后,应该立即切换成配置文件对应的配色

用户文件
weasel.custom.yaml 的配置如下:

patch:
  "style/color_scheme": google
  "style/color_scheme_dark": lost_temple
@fxliang
Copy link
Contributor

fxliang commented Dec 26, 2023

@RiwiHow
Copy link
Author

RiwiHow commented Dec 27, 2023

https://github.com/rime/weasel/actions/runs/7300873202

我是点这个链接进去后,下载的 artifact。不过依旧没有解决....

@fxliang
Copy link
Contributor

fxliang commented Dec 27, 2023

这么奇怪的嘛,你看pr里我放的动图里即便是方案里定义的配色(明暗)都可以即时切换了。。

@RiwiHow
Copy link
Author

RiwiHow commented Dec 27, 2023

但是它就是自动变不了 😭...需要我提供什么日志吗?

1

@fxliang
Copy link
Contributor

fxliang commented Dec 29, 2023

不知道是不是Windows11上没有了那个消息,留意这个产出看看情况
https://github.com/fxliang/weasel/actions/runs/7358175773

但是它就是自动变不了 😭...需要我提供什么日志吗?

1 1

@RiwiHow
Copy link
Author

RiwiHow commented Dec 30, 2023

不知道是不是 Windows11 上没有了那个消息,留意这个产出看看情况

问题解决了,感谢 👍
2

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 a pull request may close this issue.

2 participants