查看15封回复-1封至15封(共17封)
  • 插件作者 KittMedia公司

    (@kittmedia)

    你好,Bjarne,

    谢谢你把这个指给我!

    我已经将它添加到了下一个功能更新的列表中,因为我认为它不值得发布一个错误修复程序。

    顺颂商祺,
    马提亚斯

    螺纹起动器 比亚恩·奥尔德鲁普

    (@oldrup)

    我完全同意👍

    插件作者 KittMedia公司

    (@kittmedia)

    你好,Bjarne,

    我只是再看了一眼,似乎它对我来说是不可修复的。内容替换依赖于<头部>已生成。因此,我没有合适的方法在后面添加我的样式(除了一些老掉牙的手动替换)。

    我更改了CSS的加载方式,以便每个页面至少只有一个错误,而不是潜在的多个错误。但这是我目前能做的最好的事情。

    顺颂商祺,
    马提亚斯

    螺纹起动器 比亚恩·奥尔德鲁普

    (@oldrup)

    @基特米亚,非常感谢您的尝试。打得好,避免了笨拙的手动更换,这很可能会导致故障。

    我袖子里还有一个小把戏。我的缓存插件提供了将内联CSS合并到单独文件中的功能。这通常可以完成工作,尽管我在做这件事时必须注意具体问题。WordPress本身也将样式作为子项输出到div,因此如果验证对项目很重要,那么无论如何都需要注意。有道理的话,我会走那条路。感谢您抽出时间,以及这个优秀的插件。我期待着在一个真实的项目中尝试它。

    比亚内

    这里的主要问题是添加<样式>元素,而不是风格=属性。是的,拥有样式属性并不是一个好的实践,但只为一个单独的元素定义样式也没有真正的帮助,因为好的CSS类的思想是在可缓存资源中拥有可重用的类,而不是在body元素中只有一个CSS规则来处理body元素的单个元素。元素特定的内联样式是完全有效的,WordPress不强制任何CSP(事实上,许多WordPres网站在有CSP的情况下根本无法工作),那么为什么不使用它们呢?这里的性能不应该成为问题,因为每个元素都有单独的CSS类可能会产生更多的负载,因为浏览器不仅要应用样式,而且还必须首先查找它,因为它不能作为文档的一部分进行缓存。

    目前,我在自己的网站上使用分叉版本来避免无效的HTML,但您可能会考虑在未来的更新中使用内联样式。

    螺纹起动器 比亚恩·奥尔德鲁普

    (@oldrup)

    目前,我在自己的网站上使用分叉版本来避免无效的HTML,但您可能会考虑在未来的更新中使用内联样式。

    @卷心菜

    这些都是有趣的地方。它是如何工作的?您是否从这里替换正文中的样式声明(来自我的网站的示例)

    <样式>[数据嵌入id=“oembed_cd1593481ae2aa6146d42d2a21a120c4”]{纵横比:1280/720;}.embed-youtube.embed-privacy-logo{背景图像:url(https://example.net/plugins/embed-privacy/assets/images/embed-youtube.png?v=1.8.1);}</style>

    …到覆盖元素的基本内联样式(已经有一些内联样式)

    div class=“embed-privacy-container is-disabled embed-yotube”data-embed-id=“oembed_36845b8980c102d79722c9f19a826ce1”data-ebed-provider=“youtube”style=“background-image:url(https://example.net/uploads/embed-privacy/thumbnails/youtube-3GuUuHLg9t8-maxresdefault.jpg);“>

    …在这种情况下,将纵横比移动到div内联样式。

    如果转移到内联样式,这并不是很多可能重复的样式。还有,纵横比和YouTube徽标?这些可能不会在每页的基础上发生变化。如果Embed Privacy<样式>元素被禁用?也许用过滤器?

    只是在这里大声思考😊

    插件作者 KittMedia公司

    (@kittmedia)

    我只是改进了内部的样式生成f603模数转换器(另请参见改进风格生成#208).

    非常感谢您的反馈!🙂

    它生成有效的HTML,但其他任何操作都会完全失败。稍后我必须进行更彻底的测试,但当使用在提交的版本时https://github.com/ephity/embed-privacy/commit/f603adc6be1273504ab3795c1e200fb011336aa1这根本不起作用(除了不会产生无效的HTML之外)。

    JFTR:我为修复我这边的HTML输出所做的更改:https://github.com/arnowelzel/embed-privacy/commit/fec55784ce4dd0922590367775abf0998be7b2a2

    是的,我添加了所有内联样式。但如前所述,我看不出内联样式元素与内联样式所针对的元素之间有什么主要区别。

    插件作者 KittMedia公司

    (@kittmedia)

    你所说的“任何其他东西都完全失败”到底是什么意思?什么是其他东西,什么是失败的?

    只有一个盒子,里面有视频缩略图的一部分和以奇怪的方式排列的静态文本,没有任何链接可以点击。我会尝试将其截图。

    我自己的叉子:https://nextcloud.0x0c.de/s/b3rwMbPyxm29a6E/preview

    最新开发版本:https://nextcloud.0x0c.de/s/D4Yfcg6p5mL6aSR/预览

    当我有更多的时间时,我会在接下来的几天做更多的研究。

    螺纹起动器 比亚恩·奥尔德鲁普

    (@oldrup)

    哦。

    在我凌乱的测试台上测试GitHub的新开发版本:

    https://oldrup.net/embed-privacy网站/

    一些观察结果:

    • 样式元素已移动到<头部>看起来,消除HTML验证错误.
    • 管理嵌入,将状态更改为草稿,不再删除reg-exp(不同票据)
    • 已添加WordPress.tv缩略图。

    我觉得不错!

    接下来,我将在另一个站点上测试Spotify嵌入,并嵌入我的个人博客这是第三个使用不同语言的PolyLang和隐私政策的网站,这可能会引发一些奇怪的现象。

    螺纹起动器 比亚恩·奥尔德鲁普

    (@oldrup)

    下面是一篇带有Spotify嵌入内容的草稿文章的示例。是的,使用了一点自定义CSS来制作覆盖的背景色,匹配相关播客的背景色并稍微降低高度,以消除加载嵌入后的布局变化。

    还使用W3C的HTML验证器进行了测试,没有错误🥳

    插件作者 KittMedia公司

    (@kittmedia)

    感谢您的反馈!

    @奥尔德鲁普看起来你一切都很好。🎉

    @卷心菜你能测试整个开发版本吗?你可以在这里找到它:https://github.com/ephity/embed-privacy/tree/dev
    否则,我不知道怎么了。

    @基特米亚我遇到的问题https://github.com/ephity/embed-privacy/tree/dev事实上,当WordPress未在启用调试的情况下运行时,它会丢失缩小的资源。我只是没想到会这样,对不起。在复制了官方版本的缩小脚本和样式之后,这里看起来也不错。然而,我必须管理,使用输出缓冲区将样式注入head元素感觉有点奇怪。

    “…当WordPress未在启用调试的情况下运行时,它缺少缩小的资源。”

    应该是:“它缺少精简资源”。即使启用了WordPress调试,插件也会将资源排入队列。在启用调试时使用未优化的资源可能是一个好主意,如下所示:

    lightbox-photoswipe/src/LightboxPhotoSwipe/Lightbox2photoSwipe.php,见5.2.6·arnowelzel/lightbox-photoswipe(github.com)

查看15封回复-1封至15封(共17封)