🐞 反馈问题：快捷键划词翻译，窗口高度计算有误，出现大片空白

tisfeng / Easydict

一个简洁优雅的词典翻译 macOS App。开箱即用，支持离线 OCR 识别，支持有道词典，🍎 苹果系统词典，🍎 苹果系统翻译，OpenAI，Gemini，DeepL，Google，Bing，腾讯，百度，阿里，小牛，彩云和火山翻译。A concise and elegant Dictionary and Translator macOS App for looking up words and translating text.

GNU General Public License v3.0

6.45k stars 328 forks source link

🐞 反馈问题：快捷键划词翻译，窗口高度计算有误，出现大片空白 #452

Closed tisfeng closed 1 month ago

tisfeng commented 4 months ago

请先确认以下事项：

[X] 请务必查看常见问题
[X] 已仔细阅读了 README
[X] 在 issues 页面搜索过问题（包括已关闭的 issue），但未能找到解决方法
[X] Easydict 已升级到最新版本
[X] 我理解并认可上述内容，并理解项目维护者精力有限，不遵循规则的 issue 可能会被无视或直接关闭

问题描述

目测是这个 #418 PR 导致，几乎每次都会出现。

https://github.com/tisfeng/Easydict/assets/25194972/2c5ed4e9-034c-4b8c-9a5a-730bacbee1d4

该问题是否可以稳定重现？

可重现

重现步骤

截图

期望结果

解决方案（可选）

No response

设备型号 && 系统版本

是否愿意提交 PR 解决该问题？

[ ] 我愿意提交 PR

tisfeng commented 4 months ago

这个建议 https://github.com/tisfeng/Easydict/issues/404 是可以的，但感觉实现方式不是很好，之前测试偶尔会出现 tips view 一闪而过的情况。

这段代码很丑陋，嵌套逻辑太多，容易出问题。

else if ([self isTipsCellRow:row]) {
        if (!self.tipsCell) {
            height = 100;
        } else {
            height = [self.tipsCell cellHeight];
        }
    }

tisfeng commented 4 months ago

每次快捷键划词查询时，会先重置 queryText，可能会导致这里误判。

- (BOOL)isShowTipsView {
    if (Configuration.shared.disableTipsView) {
        return NO;
    }
    if (EZ_isEmptyString(self.queryModel.queryText) && 
        !self.hasShowTips &&
        self.queryModel.actionType != EZActionTypeInputQuery) {
        return YES;
    }
    return NO;
}