tongwentang / New-Tongwentang-for-Firefox

http://tongwen.openfoundry.org/
96 stars 15 forks source link

改善建议与思路(多项) #14

Open yfdyh000 opened 7 years ago

yfdyh000 commented 7 years ago
  1. 添加“选项”菜单以便于修改。我尝试制作了一版,但菜单可配置遇到问题,相关逻辑需要改善。
  2. 目前菜单是繁体在简体前面(可以理解,作者是繁体用户),这与以前及简体用户的习惯不太相符。按字母顺序也应该是Simplified在Traditional前面。菜单或应支持顺序配置(复选框?拖动调整?)或自动检测用户是简体还是繁体用户(但有些用户使用英文版。所以只能允许配置,或者按字母顺序)。
  3. “自订转换规则”的网址栏最好能自动格式化与自动填充(未查是否有相关API),以及给出例子。此外,它的布局和用词习惯也是“繁、简”。
  4. “激活右键菜单”只保留“输入区”系列,网页空白处右击将得到空的子菜单。
  5. 代码感觉有优化余地,contextMenu相关可以放在独立的模块中,并可以写Helper函数以减少常用值和结构的重复。添加菜单可以做成独立函数对象或成员数组,以便灵活配置添加顺序。
softcup commented 7 years ago

感謝你的建議也歡迎你發 pull request 給我,pull request 請發到 develop 這個 branch。

註:目前我是採用 git flow 的方式在管理 code,master branch 會維持一個乾淨可用的版本。

yfdyh000 commented 7 years ago

因为涉及contextMenus改动,所以先提出想法,以免冲突。 defaultPreference的问题,代码中有标注。version的作用不明确;新preference目前无法得到感知和填充。

softcup commented 7 years ago

了解,謝謝。 preference 目前是由 loadPreference() 來做新舊版本與使用者設定資料的處理,所以若有修改 defaultPreference 的話,卻發現沒有合併到使用者的 preference 時,可以從這邊下手。

yfdyh000 commented 7 years ago

options.js 的 init 中的 handleVelueChange 监听,是以用户当前 preferences 为准,新的 preferences 变更未包括。好像就是这个问题。其他就是架构的改进,如允许定义顺序、简化结构。

softcup commented 7 years ago

我當初的構想是,程序啟動時就做 preferences 的新舊合併,後續就使用用戶當前的 preferences。