Open
Conversation
pjchender
approved these changes
Jun 6, 2025
Owner
|
感謝，一直沒有找時間更新，但感覺我的字典檔好像還是可以更新一下 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix #3 左邊選單會凍結無法上下拖動
雖然此插件算是被 Chrome 商店半下架了，因為偶然發現這則 issue 所以嘗試進行修復。
修正後重跑 test，四項自動化測試均有通過。目前嘗試在本地 build 並套用在 issue 提及的網頁及掘金的頁面，並未發現問題。
發生原因
直接替換整頁的方式會導致部分監聽器丟失，因為該頁面的滾動行為是使用 JS 去監聽 + 實作的才會導致此情形發生
修正方式
改用遞迴替換文字節點的方式，不破壞原先的 HTML 架構。整體效能可能會變差，但應能修正此問題。