凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
阅读:12203回复:39

改进 Click to Play per-element——20140628 v0.0.5

楼主#
更多 发布于:2014-06-17 16:36
之前说过,Click to Play per-element 的作者放弃更新,不肯将 b45 转正,导致我每次都需要手动安装。不在amo也就罢了,在还这样实在有点无语。
前日无聊,干脆做个自用修改版 Click to Play per-element CE
Changelog:
2014.06.28 0.0.5版,更新内容见 https://addons.mozilla.org/zh-CN/firefox/addon/click-to-play-per-element-ce/versions/?page=1#version-0.0.5 2014.07.15 通过完全审核。
amo警告:
`pref` called in potentially dangerous manner
警告: Extensions
should not alter preferences outside of the 'extensions.' preference
branch. Please make sure that all of your extension's preferences are
prefixed with 'extensions.add-on-name.', where 'add-on-name' is a
distinct string unique to and indicative of your add-on.
当无大碍
--------------------------------------------
2014.06.26 通过 amo 初审。
--------------------------------------------
0.0.4.1
依照 AMO 建议,改善兼容性。 感谢3楼
\bootstrap.js  
_cache: Object.create(null),
---------------------------------------------
0.0.4:
\defaults\preferences\prefs.js
pref('extensions.uaSad@ClickToPlayPerElement.styles.hidePluginNotifications', true);
//默认开启隐藏插件提示条
pref('extensions.uaSad@ClickToPlayPerElement.showPluginUIEvenIfItsTooBig', true);
//默认开启显示所有插件占位符
这两个参数是之前我给作者提议他在0.0.4beta加入的,0.0.3没有,也是我这次修改的最大原因。
 
将最高支持版本提高到33.0
\install.rdf
<em:maxversion>33.0</em:maxversion>
<em:id>ClickToPlayPerElementCE@rpzrpz123.addons.mozilla.org</em:id>

最新喜欢:

文科文科 innkiinnki hzzhaiqihzzhai... fiagfiag fang5566fang55...
fang5566
管理员
管理员
  • UID3719
  • 注册日期2005-03-07
  • 最后登录2024-04-22
  • 发帖数18483
  • 经验4837枚
  • 威望5点
  • 贡献值4316点
  • 好评度1115点
  • 社区居民
  • 最爱沙发
  • 忠实会员
  • 终身成就
1楼#
发布于:2014-06-17 17:00
这个扩展我一直在用,虽然不能没通过审核,但是我还是会记得到里面装,也没什么兼容性问题。不过还是很高兴你考虑将其更新。
我也对代码不熟悉,但看样子,你不能用_proto_,要用 Object.create ,帮你找了找文档:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/create
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/proto
Firefox More than meets your experience
qweszxc
千年狐狸
千年狐狸
  • UID39616
  • 注册日期2013-06-24
  • 最后登录2022-10-14
  • 发帖数1021
  • 经验929枚
  • 威望0点
  • 贡献值1012点
  • 好评度68点
  • 社区居民
  • 忠实会员
2楼#
发布于:2014-06-17 17:36
楼主辛苦了。
hzzhaiqi
火狐狸
火狐狸
  • UID30112
  • 注册日期2009-08-18
  • 最后登录2015-09-15
  • 发帖数279
  • 经验180枚
  • 威望1点
  • 贡献值44点
  • 好评度28点
3楼#
发布于:2014-06-17 19:20
_cache: Object.create(null),

修改别人的扩展也能上架? 个人没使用同步功能,完全没动力去官方市场上架。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
4楼#
发布于:2014-06-17 19:37
hzzhaiqi:_cache: Object.create(null),
修改别人的扩展也能上架? 个人没使用同步功能,完全没动力去官方市场上架。
回到原帖
Tab Utilities Lite CE :: Firefox 附加组件
https://addons.mozilla.org/zh-CN/firefox/addon/tab-utilities-lite-customed/

主楼代码原来显示不对,现在修正了下。
hzzhaiqi
火狐狸
火狐狸
  • UID30112
  • 注册日期2009-08-18
  • 最后登录2015-09-15
  • 发帖数279
  • 经验180枚
  • 威望1点
  • 贡献值44点
  • 好评度28点
5楼#
发布于:2014-06-17 19:40
凭枰:Tab Utilities Lite CE :: Firefox 附加组件
https://addons.mozilla.org/zh-CN/firefox/addon/tab-utilities-lite-customed/

主楼代码原...
回到原帖
没懂你什么意思? Tab Utilities Lite CE 跟你主楼的不是同一个扩展,那这网址是什么意思? 没看出主楼代码哪里有修正。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
6楼#
发布于:2014-06-17 19:43
hzzhaiqi:没懂你什么意思? Tab Utilities Lite CE 跟你主楼的不是同一个扩展,那这网址是什么意思? 没看出主楼代码哪里有修正。回到原帖
我是说明修改扩展可以上架的……
原来主楼的代码挤在一行了,现在修正成本来应该的两行。
按照你说的改了下放到amo测试,警告的确没有了,多谢。
还有两个提醒:
1:本地化测试——这个我能看懂,多语言的问题,不过这个扩展不需要多语言,可以无视。
0 个错误, 0 个警告, 1 个提醒1 个提醒
Add-on cannot be localized
警告: The add-on doesn't have any locale entries in its chrome.manifest file, making it difficult to localize.

chrome.manifest
2:扩展测试——这个看不懂……
0 个错误, 0 个警告, 1 个提醒1 个提醒
`addEventListener` called with truthy fourth argument.
警告: A truthy fourth argument indicates code that accesses untrusted code. This code should be further investigated.
bootstrap.js
window.addEventListener('unload', this, false);
 gBrowser.addEventListener('PluginBindingAttached', this, true, true);
}
hzzhaiqi
火狐狸
火狐狸
  • UID30112
  • 注册日期2009-08-18
  • 最后登录2015-09-15
  • 发帖数279
  • 经验180枚
  • 威望1点
  • 贡献值44点
  • 好评度28点
7楼#
发布于:2014-06-17 20:01
凭枰:我是说明修改扩展可以上架的……
原来主楼的代码挤在一行了,现在修正成本来应该的两行。
按照你说的改了下放到amo测试,警告的确没有了,多谢。
还有两个提醒:
1:本地化测试——这个我能看懂,多语言的问题,不过这个扩展不需要多语言,可...
回到原帖
第二行有 4个 参数,删掉最后一个
gBrowser.addEventListener('PluginBindingAttached', this, true);
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
8楼#
发布于:2014-06-17 20:13
hzzhaiqi:第二行有 4个 参数,删掉最后一个
gBrowser.addEventListener('PluginBindingAttached', this, true);
回到原帖
不行,这样改就会导致功能失效。点击占位符的时候Firefox就会询问对该站点本次浏览器开启状态下允许还是永远允许,应该是默认只有本次点击允许的。
hzzhaiqi
火狐狸
火狐狸
  • UID30112
  • 注册日期2009-08-18
  • 最后登录2015-09-15
  • 发帖数279
  • 经验180枚
  • 威望1点
  • 贡献值44点
  • 好评度28点
9楼#
发布于:2014-06-17 20:45
凭枰:不行,这样改就会导致功能失效。点击占位符的时候Firefox就会询问对该站点本次浏览器开启状态下允许还是永远允许,应该是默认只有本次点击允许的。回到原帖
那不用去理这个警告。第四个参数只用于Gecko/Mozilla,其它浏览器不行,所以会有这个警告。详见 https://developer.mozilla.org/zh-CN/docs/DOM/element.addEventListener
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
10楼#
发布于:2014-06-17 20:59
hzzhaiqi:那不用去理这个警告。第四个参数只用于Gecko/Mozilla,其它浏览器不行,所以会有这个警告。详见 https://developer.mozilla.org/zh-CN/docs/DOM/element.addEventListene...回到原帖
原来如此,多谢!
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
11楼#
发布于:2014-06-26 09:45
fang5566:这个扩展我一直在用,虽然不能没通过审核,但是我还是会记得到里面装,也没什么兼容性问题。不过还是很高兴你考虑将其更新。
我也对代码不熟悉,但看样子,你不能用_proto_,要用 Object.create ,帮你找了找文档:
https:...
回到原帖
2014.06.26 通过 amo 初审。
fang5566
管理员
管理员
  • UID3719
  • 注册日期2005-03-07
  • 最后登录2024-04-22
  • 发帖数18483
  • 经验4837枚
  • 威望5点
  • 贡献值4316点
  • 好评度1115点
  • 社区居民
  • 最爱沙发
  • 忠实会员
  • 终身成就
12楼#
发布于:2014-06-26 09:58
自己动手,丰衣足食啊,支持一下,上架一定下载使用。
建议增加一个原来的功能:隐藏那个信息栏:
@namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
notification[value="plugin-hidden"]{
    display:none !important;
}
Firefox More than meets your experience
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
13楼#
发布于:2014-06-26 12:25
fang5566:自己动手,丰衣足食啊,支持一下,上架一定下载使用。
建议增加一个原来的功能:隐藏那个信息栏:
@namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.on...
回到原帖
是地址栏www前面那个小图标?
之前哪个版本存在的,我去看下……
fang5566
管理员
管理员
  • UID3719
  • 注册日期2005-03-07
  • 最后登录2024-04-22
  • 发帖数18483
  • 经验4837枚
  • 威望5点
  • 贡献值4316点
  • 好评度1115点
  • 社区居民
  • 最爱沙发
  • 忠实会员
  • 终身成就
14楼#
发布于:2014-06-26 12:28
凭枰:是地址栏www前面那个小图标?
之前哪个版本存在的,我去看下……
回到原帖
就是工具栏下方会显示一栏问你允许还是阻止。之前论坛又帖子讨论,这个扩展之前版本ok,后来失效,我现在单独用css隐藏。
Firefox More than meets your experience
上一页
游客

返回顶部