search

benzBrake / FirefoxCustomize

Ryan 收集的 Firefox 个性化相关资源
153 stars 34 forks source link

README 两个链接错误 #29

Closed wordpure closed 6 months ago

wordpure commented 6 months ago
  1. userChromeJS - 脚本列表的 autoPopup++ 链接错了，后缀从 .js 变成了 .s。

图片

  1. 扩展选项菜单 的链接连到后面的文字了。

图片

wordpure commented 6 months ago

顺带一提，大佬有没有兴趣统一下 usrchrome 配置项的前缀？感觉统一加上 userChromeJS 后续查找的时候会方便一些

图片

benzBrake commented 6 months ago

顺带一提，大佬有没有兴趣统一下 usrchrome 配置项的前缀？感觉统一加上 userChromeJS 后续查找的时候会方便一些

图片

前面两个没有 userChromeJS 开头是因为历史原因，最原始作者版本的时候就这样了

wordpure commented 6 months ago

前面两个没有 userChromeJS 开头是因为历史原因，最原始作者版本的时候就这样了

如果不麻烦的话，可以加个判断来实现向后兼容，规范统一点总是好的

wordpure commented 6 months ago

Markdown 里单纯的链接可以直接用尖括号包裹来着。

图片

图片

benzBrake commented 6 months ago

Markdown 里单纯的链接可以直接用尖括号包裹来着。

图片

图片

我使用 Typora 编写的，具体 markdown 我不编辑的

benzBrake commented 6 months ago

前面两个没有 userChromeJS 开头是因为历史原因，最原始作者版本的时候就这样了

如果不麻烦的话，可以加个判断来实现向后兼容，规范统一点总是好的

历史遗留导致很多 prefs 都是存成 addMenu 开头的，如果别人有存为addMenu开头的情况下不好兼容读取

https://github.com/benzBrake/FirefoxCustomize/blob/620b4a158406b6477c934d96f05c2130bb13c899/userChromeJS/addMenuPlus/addMenuPlus_fx72.uc.js#L428-L431

有需要的话你自己改动一下改成userChromeJS.addMenu.开头

wordpure commented 6 months ago

历史遗留导致很多 prefs 都是存成 addMenu 开头的，如果别人有存为addMenu开头的情况下不好兼容读取

我还以为只有这一个设置项是 addMenu 开头的，可以直接改成下面这样😂

图片