-
Notifications
You must be signed in to change notification settings - Fork 217
apply theme to TextEditingUi + editable clipboard content radius #489
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
app/src/main/java/org/fcitx/fcitx5/android/input/keyboard/KeyDef.kt 64行,radius未传值,93行同理
app/src/main/java/org/fcitx/fcitx5/android/input/popup/PopupComponent.kt
Outdated
Show resolved
Hide resolved
加了 |
禁用按键逻辑 与 Gboard 一致 show.mp4 |
希望不要影响类似termux这种应用中的表现,例如上下切换history等 |
这个 Select 键还有些问题,原本的实现是,只有在编辑器中有选择的文字时,才会变成蓝色(accentKeyBackgroundColor),取消选择后,就会变回普通按键的颜色 |
app/src/main/java/org/fcitx/fcitx5/android/input/keyboard/KeyDef.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fcitx/fcitx5/android/input/editing/TextEditingUi.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fcitx/fcitx5/android/input/editing/TextEditingUi.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/org/fcitx/fcitx5/android/input/editing/TextEditingUi.kt
Outdated
Show resolved
Hide resolved
这个改好了 |
d8a3dd7
to
ad31556
Compare
@rocka 现在我加了选项,3种按钮随便选,每个人应该都满意了,你可以试试 |
365a08f
to
e3d773f
Compare
我下载 Release版 0.0.9 确认了下,之前线条效果的文本编辑按键有个bug,就是在我的平板上,这些线条是不可见的 |
223493f
to
ecfc97e
Compare
首先我得先道个歉,最近确实比较忙,没时间处理 PR ,尤其是这种规模比较大的 ... 然后就要开始挑刺儿了。 虽然之前提交的代码“看起来”达成了描述中的效果,但落到实现细节上,有些地方对已有封装类的使用有误,破坏了整体的抽象层次。比如,在 fcitx5-android/app/src/main/java/org/fcitx/fcitx5/android/input/editing/TextEditingUi.kt Lines 53 to 60 in ecfc97e
和 fcitx5-android/app/src/main/java/org/fcitx/fcitx5/android/input/keyboard/KeyDef.kt Lines 19 to 36 in ecfc97e
中使用 其次,对 Splitties View DSL 中 fcitx5-android/app/src/main/java/org/fcitx/fcitx5/android/input/picker/PickerPageUi.kt Lines 48 to 56 in ecfc97e
中就不应该将 另外,没必要每种实现都加个选项,让符号键盘和文本编辑界面都跟随键盘的按键边框设置就够了,也没必要每个边距都独立成一个设置,否则设置界面的膨胀速度就不可控了。 最后,每个 PR 应该只包含一个功能的实现,不应该把多个互不相关的功能放在一起,否则会给 review 增加不必要的工作量(要额外区分哪些文件实现了哪些功能),也不方便合并的时候直接 squash 。 代码我已经全都重构成比较符合目前编码风格的实现了,如果有问题可以再商量,没问题的话就按照现在的实现合并了(可能还需要再 rebase 一下,把相关的功能 squash 成一个 commit)。 |
@rocka 一切按照你改的来,这代码我改起来头疼,最近也比较忙 |
Co-authored-by: cc <ccyybn@gmail.com>
Co-authored-by: cc <ccyybn@gmail.com>
Co-authored-by: cc <ccyybn@gmail.com>
Text Editing UI
Symbol Key Border
Clipboard Content Radius