trime icon indicating copy to clipboard operation
trime copied to clipboard

Rewrite ascii mode switch logic

Open nopdan opened this issue 2 years ago • 3 comments

Pull request

重写 ascii_mode 切换逻辑,记住每个键盘布局的 ascii_mode

Issue tracker

Fixes will automatically close the related issues

Fixes #1092 Fixes #1028 Fixes #1035

Feature

Describe features of this pull request

Code of conduct

  • [x] CONTRIBUTING

Style lint

  • [x] make sytle-lint

Build pass

  • [x] make debug

Manually test

  • [x] 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

nopdan avatar Feb 07 '24 08:02 nopdan

image image

@goofyz 我用虚拟机测试的,确实是非常慢,你试一下编译为 release 版本的速度吧

nopdan avatar Feb 07 '24 09:02 nopdan

release 版平均也要 200ms

goofyz avatar Feb 08 '24 09:02 goofyz

release 版平均也要 200ms

我这边真机 debug 都只需要 5ms,WP 使用虚拟机+debug 也只要几十ms,可能还是手机的问题。

这个 pr 只是修复 ascii_mode 切换 bug。

以后可以写个专门的符号键盘,不通过 rime 直接上屏,可以避免这个问题。

nopdan avatar Feb 08 '24 12:02 nopdan

@goofyz 再看看吧,逻辑应该没问题了,性能我没法解决

nopdan avatar May 07 '24 07:05 nopdan

在我的機子是慢了。不過也可以接受。 @WhiredPlanck @Bambooin 你們的有沒有問題?

goofyz avatar May 08 '24 07:05 goofyz

@goofyz @nopdan 我这边第一次冷启动会卡一下,其余时间相对正常。

WhiredPlanck avatar May 08 '24 10:05 WhiredPlanck