Skip to content
This repository has been archived by the owner on Apr 3, 2021. It is now read-only.

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

Open
yfdyh000 opened this issue May 24, 2017 · 5 comments
Open

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

yfdyh000 opened this issue May 24, 2017 · 5 comments

Comments

@yfdyh000
Copy link
Contributor

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

softcup commented May 24, 2017

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

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

@yfdyh000
Copy link
Contributor Author

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

@softcup
Copy link
Collaborator

softcup commented May 24, 2017

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

@yfdyh000
Copy link
Contributor Author

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

@softcup
Copy link
Collaborator

softcup commented May 25, 2017

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants