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: customize the font size #268

Merged
merged 22 commits into from
Dec 22, 2023
Merged

Conversation

NeverAgain11
Copy link
Collaborator

@NeverAgain11 NeverAgain11 commented Dec 18, 2023

close #137
close #237

@tisfeng tisfeng changed the title Change font size feat: customize the font size Dec 18, 2023
@tisfeng
Copy link
Owner

tisfeng commented Dec 18, 2023

@Jerry23011 @choykarl 你们先看一下代码,运行看看效果。

@Jerry23011
Copy link
Collaborator

我这边看 Bing 词典里面的一些元素字体没有变化
截屏2023-12-18 11 06 04

@choykarl
Copy link
Collaborator

我这边看 Bing 词典里面的一些元素字体没有变化 截屏2023-12-18 11 06 04

@NeverAgain11 这个是在EZWordResultViewbuildSynonymsAndAntonymsView:方法里。如果把设置字体比例放在EZBlueTextButton setFontSize: 方法里,这里就不需要修改,之后其他地方用在这个button也不需要关心比例的问题。

@NeverAgain11
Copy link
Collaborator Author

@Jerry23011 @choykarl 这个字体大小改了

@tisfeng
Copy link
Owner

tisfeng commented Dec 19, 2023

简单看了一下,整体没什么问题,但有些细节需要优化一下:

  • 服务图标右侧的标题,这个字体暂定不改,保持和图标协调。
  • “好处; 好人; 益处; 善行” 这个字体,我记得之前是没有加粗的?
  • Bing 翻译部分字体没有适配,蓝色按钮,还有后面词组的解释部分。
  • 字体变化范围太大了,默认的字体应该就是最小值,不应该再小了,最大值太大了,需要调小一点。
image

@tisfeng
Copy link
Owner

tisfeng commented Dec 19, 2023

这里可以加一个支持拖动的手势,这样更符合使用习惯。

image

@tisfeng
Copy link
Owner

tisfeng commented Dec 19, 2023

另外,有时候这个表示当前字体大小的浮标不见了

image

@tisfeng
Copy link
Owner

tisfeng commented Dec 19, 2023

我简单修改了几点,其他的你们再看看。

@tisfeng
Copy link
Owner

tisfeng commented Dec 20, 2023

可以,目前效果已经很不错了,可以参考这个,再稍微优化一下:

  • 左右添加一个描述字体大小的标识。
  • 增加快捷键 Cmd +/- 修改字体大小,参考 Pin 的快捷键设置,绑定一下事件就可以。
image
image image

@tisfeng
Copy link
Owner

tisfeng commented Dec 20, 2023

补充了苹果词典中 iframe body 字体大小修改。

@tisfeng
Copy link
Owner

tisfeng commented Dec 21, 2023

快捷键 Shortcuts 菜单项部分默认是隐藏的,一般用户可能注意不到,因此可以在设置页下面加一下支持快捷键修改字体大小的提示。

image

@NeverAgain11
Copy link
Collaborator Author

提示已加

@tisfeng
Copy link
Owner

tisfeng commented Dec 22, 2023

没问题了,感谢你的 PR,欢迎后面再来 PR 😃

@tisfeng tisfeng merged commit a930ec4 into tisfeng:dev Dec 22, 2023
2 of 3 checks passed
@tisfeng
Copy link
Owner

tisfeng commented Dec 23, 2023

@NeverAgain11 你好,请问有兴趣一起来参加开发和维护 Easydict 吗? 处理 issue,Swift 重构,或者开发一些新的功能都可以。

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.

4 participants