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

feat: 添加 style/comment_on_top #1471

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

feat: 添加 style/comment_on_top #1471

wants to merge 6 commits into from

Conversation

ijklam
Copy link

@ijklam ijklam commented Jan 9, 2025

feat: 添加 style/comment_on_top,使得在水平布局时,拼音等注释可以显示在候选字之上。(类似于 Android 同文输入法的效果)

效果如下:

图片

图片

@lotem
Copy link
Member

lotem commented Jan 10, 2025

牛的。

我看好你。不過還是要梁老師來把關。

@lotem lotem requested a review from fxliang January 10, 2025 06:39
@ijklam ijklam marked this pull request as draft January 10, 2025 16:27
@fxliang
Copy link
Contributor

fxliang commented Jan 11, 2025

感谢努力,我看到的几个点
1,这个看上去通用的选项,却只对其中一个布局影响了,不是太合理
2,这样调整了之后,原来的一些布局参数就更难理解在不同情况下的作用了。比如align_type(手机敲的希望没记错),hilite_spacing, candidate_spacing在不同布局下再加上这个on top的参数的作用会变得相当不一致,非常难解析清楚,要再考虑调整下。
3,如果是要实现布局的更灵活的配置,其实我早前有个脑洞上先算完全部候选的三个部件的size,如果某些指定路径下有脚本的话就export到lua(或其他)中,用户自行定义方式计算返回rects等。
4,个人判断而言,这样布局出来的文字部分居中部分不是的状态稍乱,不够美观。

@ijklam
Copy link
Author

ijklam commented Jan 11, 2025

1,这个看上去通用的选项,却只对其中一个布局影响了,不是太合理

先搞定一个比较好

2,这样调整了之后,原来的一些布局参数就更难理解在不同情况下的作用了。比如align_type(手机敲的希望没记错),hilite_spacing, candidate_spacing在不同布局下再加上这个on top的参数的作用会变得相当不一致,非常难解析清楚,要再考虑调整下。

确实

3,如果是要实现布局的更灵活的配置,其实我早前有个脑洞上先算完全部候选的三个部件的size,如果某些指定路径下有脚本的话就export到lua(或其他)中,用户自行定义方式计算返回rects等。

这个好。直接改 C++ 然后编译安装重启这一套流程太麻烦了,还好有虚拟机。

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