凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
阅读:12273回复: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...
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
1楼#
发布于: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/

主楼代码原来显示不对,现在修正了下。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
2楼#
发布于: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);
}
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
3楼#
发布于:2014-06-17 20:13
hzzhaiqi:第二行有 4个 参数,删掉最后一个
gBrowser.addEventListener('PluginBindingAttached', this, true);
回到原帖
不行,这样改就会导致功能失效。点击占位符的时候Firefox就会询问对该站点本次浏览器开启状态下允许还是永远允许,应该是默认只有本次点击允许的。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
4楼#
发布于: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点
  • 社区居民
  • 忠实会员
5楼#
发布于:2014-06-26 09:45
fang5566:这个扩展我一直在用,虽然不能没通过审核,但是我还是会记得到里面装,也没什么兼容性问题。不过还是很高兴你考虑将其更新。
我也对代码不熟悉,但看样子,你不能用_proto_,要用 Object.create ,帮你找了找文档:
https:...
回到原帖
2014.06.26 通过 amo 初审。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
6楼#
发布于:2014-06-26 12:25
fang5566:自己动手,丰衣足食啊,支持一下,上架一定下载使用。
建议增加一个原来的功能:隐藏那个信息栏:
@namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.on...
回到原帖
是地址栏www前面那个小图标?
之前哪个版本存在的,我去看下……
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
7楼#
发布于:2014-06-26 12:31
fang5566:就是工具栏下方会显示一栏问你允许还是阻止。之前论坛又帖子讨论,这个扩展之前版本ok,后来失效,我现在单独用css隐藏。回到原帖
原版有这功能的就是默认关闭,我的修改有一点就是默认开启这个功能。
我想把Firefox的提示插件图标设为logo,问题是那个图片怎么提取出来?
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
8楼#
发布于:2014-06-28 14:44
qweszxc:楼主你能否详细说下你这个自用修改版 Click to Play per-element CE 的和原来的Click to Play per-element 0.0.3.20130818最新正式绝版的具体有什么区别?或就是说你具体需要增加了什...回到原帖

详细更新日志见主楼。

作者给我发了封邮件:

I just want to say that the
        same name can lead to conflicts if the user installs both additions.
 For example:
 
 chrome.manifest:content uasadclicktoplayperelement ./
 bootstrap.js:const PREF_BRANCH =
          'extensions.uaSad@ClickToPlayPerElement.';
 const PREF_FILE =
'chrome://uasadclicktoplayperelement/content/defaults/preferences/prefs.js';defaults/preferences/prefs.jsall prefs
 
 and maybe something else.
所以我改了下0.0.5,还请帮忙测试
click_to_play_per_element_ce-0.0.5.1-fx.xpi
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
9楼#
发布于:2014-06-28 17:15
fang5566:哈,已经看到正式上架了,立马安装了,卸载原版。回到原帖
18楼0.0.5版,详细更新日志见主楼。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
10楼#
发布于:2014-06-29 16:45
qweszxc:用24esr测试过了2014.06.28 0.0.5版(click_to_play_per_element_ce-0.0.5.1-fx.xpi)赶脚不错,另外乃自用修改版 Click to Play per-element CE网站里面 “...回到原帖
提交0.0.5的时候一并修改。
帮我看下主楼更新的amo的警告吧。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
11楼#
发布于:2014-06-29 18:49
fang5566:那个警告只是说 pref.js 里面的你扩展的选项要是 extensions.附加组件名称.这样打头回到原帖
那我添加了那些参数,能实现同步么?
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
12楼#
发布于:2014-06-29 23:04
fang5566:没试过,看过去写的没错,应该可以同步的回到原帖
嗯,谢谢,提交到amo了,排名138,估计要审核好久……
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
13楼#
发布于:2014-12-05 21:56
fang5566:作者更新很久了,早已支持最新版本了。楼主可以考虑跟进回到原帖
0.06出来就知道了,只支持35及以上,就没管,才发现0.07是全兼容了。
如果原作者的版本可以的话,我也没必要自己维护一个版本了吧。自己搞一个单纯是因为原作者不更新不转正没法同步。
明天有空仔细看下。
凭枰
狐狸大王
狐狸大王
  • UID32865
  • 注册日期2010-05-17
  • 最后登录2021-08-29
  • 发帖数536
  • 经验489枚
  • 威望1点
  • 贡献值226点
  • 好评度50点
  • 社区居民
  • 忠实会员
14楼#
发布于:2014-12-06 00:07
fang5566:隐藏插件提示栏是你这版本的亮点啊回到原帖
就是默认开启了个参数……
上一页
游客

返回顶部