使WordPress成为核心

开的3年前

上次修改时间3周前

#52294 新的 增强

add_editor_style:允许用RTL版本替换样式

报告人: 尤夫的个人资料 尤亚夫 所有者:
里程碑: 6.7 优先: 正常的
严重程度: 正常的 版本: 5.6
组件: 主题 关键词: has-patch接口 需求-测试 has-unit-测试
重点: 实时通讯 复写的副本:

描述(上次修改者尤亚夫)

Twenty Twenty-One中的编辑器样式没有RTL版本。
在RTL模式下,这会导致编辑器中出现重大样式问题。

例如,嵌套有序列表如下所示:

请注意,所有项目都位于同一水平面上。

这是主题编辑器样式关闭时的外观:

问题的详细信息

目前添加编辑器样式RTL支持非常有限。如果文件存在,它将加载样式编辑器-rtl.css文件,此外到主管道样式编辑器.css文件(类似于为主题加载rtl.css的方式)。

这个方法的问题是,我们目前没有生成插件rtl文件的工具,该文件只覆盖了必要的CSS声明。

复杂度为二十一样式编辑器.css文件(约2500行,SASS生成),创建手动附加文件是不合理的。

可能的解决方案

任何解决方案都应该从生成完全镜像的样式编辑器-rtl.css使用rtlcss自动创建文件。

我考虑了几个可能的解决方案,我很高兴听到大家对此的意见:

  1. 我们可以在二十世纪封锁编辑风格-它没用添加编辑器样式而是打电话wp_排队_样式,然后使用wp样式添加数据定义RTL版本应代替ltr版本。
  1. 我们可以更新添加编辑器样式以支持加载附加参数。我附上了一个如何做到这一点的例子。

附件(3)

method-2示例.diff(3.0 KB)-由添加尤亚夫 3年前.
将新参数添加到add_editor_style以允许用RTL版本替换css文件
编辑器样式-broken.png(54.3 KB)-由添加约阿夫 3年前.
编辑器样式-禁用.png(45.6 KB)-由添加约阿夫 3年前.

将所有附件下载为:.zip文件

更改历史记录(51)

@尤亚夫
3年以前

将新参数添加到add_editor_style以允许用RTL版本替换css文件

#1 @尤亚夫
3年以前

  • 版本设置为5.6

#2 @尤亚夫
3年以前

#3 @谢尔盖·比尤科夫
3年以前

  • 组件已从更改主题捆绑主题

#4 随访: @高兴地
3年以前

另一种选择是样式表同时包含LTR和RTL样式。我在我的主题中这样做。当它们在一起时,可以更容易地保持样式的一致性。

#5 答复: 4 @尤亚夫
3年以前

回复高兴地:

另一种选择是样式表同时包含LTR和RTL样式。我在我的主题中这样做。当它们在一起时,保持风格的一致性会容易得多。

这是可行的(通过将.rtl主体类作为目标),但您有在主样式表中自动生成rtl CSS的解决方案吗?

#6 @波纳
3年以前

我喜欢为所有主题提供新参数的想法。

我搜索了Trac,看看是否有重复的内容。我没有找到但是
也许它可以与#42645,支持将版本号传递给add_editor_style()。

上次编辑时间3年前通过谢尔盖·比尤科夫(以前的)(差异)

这张票是在松弛(Slack)在poena的《themereview》中。查看日志.


3年以前

#8 @波纳
3年以前

  • 关键词 has-patch接口 需要测试补充
  • 里程碑已从更改等待审查5.9

#9 @萨伯恩哈特
3年以前

  • 总结已从更改Twenty-Twentyone:RTL中的编辑器样式被破坏Twenty Twenty-One:RTL中的编辑器样式被打破

这张票是在松弛(Slack)在kafleg的《themereview》中。查看日志.


3年以前

这张票是在松弛(Slack)chaion07的in#core。查看日志.


3年以前

这张票是在松弛(Slack)在hellofromtonya的核心测试中。查看日志.


3年以前

#13 @chaion07号
3年以前

嗨@yoavf!非常感谢您的报道。我们在最近的一次错误扫描中讨论了此问题,并根据讨论添加了“测试说明”。它还可以使用关键字“Add_Editor_Style”(目前正处于正式添加关键字的建议阶段)而不是“Theme”。此外,考虑到测试团队计划明天主持的手动测试会话,最好将此添加到议程中。测试报告可用后,应处于“提交”的理想位置。干杯!

道具:@hellofromTonya&@poena

这张票是在松弛(Slack)boniu91进行的#核心测试。查看日志.


3年以前

这张票是在PR#1815WordPress/WordPress-develop开发通过伯恩.


3年以前
#15

  • 关键词 has-unit-测试补充

在add_editor_style函数中添加了用RTL版本替换样式表的支持

为twentytyOne主题添加了rtl编辑器样式表
添加了add_editor_style测试

Trac票:https://core.trac.wordpress.org/ticket/52294

#16 @伯恩
3年以前

  • 关键词 has-patch接口 需求-测试 has-unit-测试远离的

大家好,

我必须更改代码以获得add_editor_style函数以使其工作

我已经测试了rtl样式表是否加载到经典编辑器(tinyMCE)中

很好,可以走了

保罗

这张票是在松弛(Slack)pbearne进行的#核心测试。查看日志.


3年以前

#18 @hellofrom托尼亚
3年以前

  • 关键词 has-patch接口 需求-测试补充
  • 里程碑已从更改5.96

由于5.9 Beta 1将在几个小时内发布,因此将此问题移至6.0。然而,如果它能够得到测试,并且在发布方之前就补丁达成了共识,那么就把它移回里程碑。

#19 @坡那
2年以前

我无法重现列表块缺少缩进的原始示例。

当我尝试使用npm运行生成在pull请求中,我收到了阻止构建的警告。
也许它需要重新设置基址?

贾斯汀·阿欣对发表了评论PR#1815:


2年以前
#20

您好@pbearne,请您在PR中添加一些测试说明好吗?

伯恩对发表了评论PR#1815:


2年以前
#21

测试:
安装从右到左的语言
打开编辑器并注意到布局与前端匹配

应用补丁和布局应该匹配,因为我们现在不加载ltr CSS,只加载rtl CSS

这张票是在松弛(Slack)迈克的《核心》。查看日志.


2年以前

这张票是在松弛(Slack)在迈克的《核心编辑》中。查看日志.


2年以前

这张票是在松弛(Slack)在迈克的核心主题中。查看日志.


2年以前

#25 @基拉松
2年以前

我们看了这张票在除虫过程中今天。
我查过了,公关部的补丁仍然适用。

我联系了#核心编辑和#核心主题寻求反馈。

#26 @hellofrom托尼亚
2年以前

  • 关键词 has-unit-测试补充

6.0的最后一个测试版将在15分钟内完成,明天是RC1。虽然已经接近准备就绪,但还需要更多的测试和审查。由于这个问题没有在6.0中引入,而且还没有完全准备好提交,因此将其转移到下一个周期。谢谢大家的贡献!

#27 @成本开发
2年以前

  • 里程碑已从更改66.1

遗憾的是,这是如此接近,但没有得到注意这个周期。虽然它已经经历了几个周期,因为它已接近解决,但我将把这张票移到6.1里程碑。让我们尝试在6.1中着陆!

#28 @坡那
2年以前

我现在已经能够建立公共关系了。
测试说明仍有点模糊:
当我添加问题描述中的嵌套列表块时,该块在LTR和RTL中看起来都正确:当我应用PR时,此布局不会更改。

相反,我做了一个更有限的测试,在主题的assets/css/style-editor-rtl.css文件中添加了一个红色边框样式。使用此样式,我能够确认样式正在加载到编辑器中。

我还可以确认文件的生成工作正常(将左替换为右,npm命令工作正常)。

MacO 12.4款
铬02.0.5005.115
WordPress 6.0语言

上次编辑时间2年前通过波纳(以前的)(差异)

#29 @萨伯恩哈特
21个月以前

  • 里程碑已从更改6.16.2

#30 @阿德塔赫里
17个月以前

测试报告

测试补丁:NO补丁

环境

  • 操作系统:macOS 13.0.1
  • Web服务器:Nginx
  • 菲律宾比索:7.4.33
  • WordPress:6.2-字母54642-src
  • 浏览器:Chrome 109.0.5414.125
  • 主题:二十二二十一
  • 活动插件:
    • RTL测试仪

实际结果

RTL后端:网址:https://d.pr/i/5VDLl6
RTL前端:https://d.pr/i/pnHmES网站
LTR后端:https://d.pr/i/vBkpYS网站
轻轨前端:https://d.pr/i/yr63Ma网站

  • 问题在没有修补程序的情况下得到解决。

#31 @波纳
17个月以前

  • 关键词 需要测试信息补充

#32 @波纳
17个月以前

  • 关键词 需求-测试-信息远离的

@pbearne和@costdev请购单上的所有变更都解决了吗?
如果是的话,我想这已经准备好提交了吗?
主题CSS文件可能需要重建,

我相信执行情况很好,只是《Twenty-Twenty-One》中列表的原始版本无法复制。

上次编辑时间17个月前通过波纳(以前的)(差异)

#33 @波纳
17个月以前

我们应该将组件从捆绑主题更改为编辑器?

#34 @罗宾开发者
17个月以前

是的,我看到嵌套列表问题是不可复制的。
无法复制。

屏幕截图:
RTL编辑器:https://d.pr/i/EGJpIF网站
RTL前端:https://d.pr/i/lwAugw(网址:https://d.pr/i/lwAugw)

#35 @波纳
17个月以前

  • 总结已从更改Twenty Twenty-One:RTL中的编辑器样式被打破add_editor_style:允许用RTL版本替换样式

这张票是在松弛(Slack)在costdev的#core中。查看日志.


16个月以前

#37 @弗利克索斯90
16个月以前

@poena@pbearne@costdev@robinwpdeveloper虽然此问题最初源于某种破碎的体验,但我不同意此处使用的当前问题类型:技术上,这显然是一个增强,因为它为主题常用的函数添加了一个新参数。在6.2中解决这个问题也没有特别的紧迫性,因为它并没有在这个版本中引入,事实上多年来一直是这样。

虽然我完全同意应该解决这个问题,但对我来说,在6.2发行周期的后期,我们不应该承诺这一点,所以我强烈建议将其提交到6.3,并在该版本中提交。

#38 @成本开发
16个月以前

  • 里程碑已从更改6.2未来版本
  • 类型已从更改缺陷(bug)增强

很高兴这么做@flixos90,谢谢!

重新分类为增强并移至未来版本因为这张票已经过了几个周期。

#39 @波纳
5个月以前

  • 里程碑已从更改未来版本6.5

#40 @波纳
5个月以前

  • 组件已从更改捆绑主题主题

#41 @伯恩
5个月以前

刷新的修补程序

#42 @瑞士风格
4个月以前

  • 里程碑已从更改6.5未来版本

#43 @波纳
2个月以前

  • 里程碑已从更改未来版本6.6

这张票是在松弛(Slack)in#core由nhrrob提供。查看日志.


5周以前

这张票是在松弛(Slack)由nhrrob进行#核心测试。查看日志.


5周以前

这张票是在松弛(Slack)oglekler的in#core。查看日志.


4周以前

#47 @奥格勒克勒
4周以前

在4个月内没有实际行动,但由于@poena,我将把这张票移到6.7里程碑,而不是未来版本。

#48 @奥格勒克勒
3周以前

  • 里程碑已从更改6.66.7
注:请参见TracTickets公司有关使用的帮助门票。