Lin Zhi-xin

Results 47 comments of Lin Zhi-xin

没必要。按这样的话那不是所有带θ的都要列进去了?这个仓库又不是教音标发音的。

JSON、HTTP、HTML这种还要翻译成中文就别想有人用了。 这个我觉得有很简单的评判标准,中文技术圈在写教程、日常交流时时习惯怎样交流就用什么。

我想到有一种可能容易忽略的情况。 浏览器挂着 Learn Helper 的 tab,使用 Windows 的「休眠」关闭了系统。再次启动系统进入 Learn Helper 时,基本都会出现「未登录」,这个我遇到过很多次。 如果只是定时调用登录 API 的话,似乎会漏掉这种情况?因为 Learn Helper 没有一直在运行。

> > 我想到有一种可能容易忽略的情况。 > > 浏览器挂着 Learn Helper 的 tab,使用 Windows 的「休眠」关闭了系统。再次启动系统进入 Learn Helper 时,基本都会出现「未登录」,这个我遇到过很多次。 > > 如果只是定时调用登录 API 的话,似乎会漏掉这种情况?因为 Learn Helper 没有一直在运行。 > > 是否可能可以改成:全局储存上次登录的时间,在所有操作(比如下载课程文件/作业附件、打开提交作业的IFrame的时候),检查保存的上次登录时间,发现超过一定阈值就静默登录后再执行操作? 相当于是一种懒登录的模式 我认为可行。

或者也可以在捕获到「未登录」问题的时候自动调用“尝试登录”或“手动刷新”?这样的话不需要添加额外的数据。

> 似乎只会在 14 分钟后运行一次? 会不会是所依附的窗口被关闭了?我对项目的代码结构不是很了解,可能定时函数不应写在这个地方。我之前只用过 Vue,没用过 React,写这个修改只是想尝试一下,可以不必太在意我的实现方式。 另外在 issue 的讨论里也考虑到定时调用 API 可能不是最完善的方案。

也许可考虑默认就不允许时间冲突?因为如果冲突的话,似乎目前并没有足够妥善的解决方案。

我会倾向于直接让冲突的时间不可选择。

你说得对。那把「不可选择」改成「弹窗询问 “覆盖原课程”or“取消选择”」会更妥当些。 话说现在是只能选择隐藏整个课程,没法部分覆盖吗……