跳到内容
新问题

对这个项目有疑问吗?注册一个免费的GitHub帐户以打开一个问题,并联系其维护者和社区。

单击“注册GitHub”,表示您同意我们的服务条款隐私声明。我们偶尔会向您发送与帐户相关的电子邮件。

已经在GitHub上了?登录到您的帐户

WP_HTML_Tag_Processor:支持标签关闭书签 #第4115页

对话

阿达齐尔
复制链接
贡献者

@阿达齐尔 阿达齐尔 评论2023年2月22日

描述

使用此PR搜索()正确查找标签关闭器上设置的书签。它:

  • 使用正确的起始索引标记更紧密的书签
  • 添加数组('tag_closers'=>'visit')在里面搜索()

关于书签开始索引:

在标签上设置书签时,应在开始“<”之前设置其“开始”位置,例如:

测试a书签</b>----------------^

当前计算假定这始终是$tag_name_starts_at左边的一个字节。

然而,在索引指向索利多金币符号“/”的标签闭合器中:

测试a书签</b>----------------------------^

因此,书签应该在标记名之前的两个字节开始:

测试a书签</b>---------------------------^

Trac票:https://core.trac.wordpress.org/ticket/57787网址

复写的副本@奥卡姆 @dmsnell公司 @吉奥罗牌手表

…更接近标签此提交在一个字节之前标记书签的开始标记打开器的标记名以两个字节开头标记闭合器的标记名。在标记上设置书签应设置其“开始”位置在打开“<”之前,例如:```测试a书签</b>----------------^```当前计算假设这始终是一个字节从$tagnamestarts_at向左。然而,在标签闭合器中,索引指向索利多金币符号“/”:```测试a书签</b>----------------------------^```因此,书签应该在标记名之前的两个字节开始:```测试a书签</b>---------------------------^```
复制链接
贡献者

@希腊罗马帝国 希腊罗马帝国 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

  • 申请前WP_HTML_Tag_处理器更改,测试失败🔴
  • 应用后WP_HTML_Tag_处理器更改,测试通过🟢

这些更改的工作原理如下所述准备提交👍

复制链接
成员

@费利萨尔茨 费利萨尔茨 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

@阿达齐尔LGTM也是!

复制链接
贡献者

@dmsnell公司 dmsnell公司 留下了评论

选择隐藏此评论的原因

将显示原因,以便向其他人描述此评论。了解更多信息.

看起来不错。谢谢@阿达齐尔

@希腊罗马帝国
复制链接
贡献者

通过提交https://core.trac.wordpress.org/changeset/55407.

奥卡姆添加了提交到WordPress/gutenberg引用的这个拉请求 2023年3月2日
在6.2 compat层中,支持指向关闭标记的书签。这是为了与WP 6.2中的Core对等,请参见WordPress/WordPress-develop#4115和[55407兰特](https://core.trac.wordpress.org/changeset/55407).请注意,这些更改已在中回传到GB的6.**3**compat层#48378但是,由于更改将在WP 6.2中进行,因此我们还必须将其应用于GB的6.2 compat层。
奥卡姆添加了提交到WordPress/gutenberg引用的这个拉请求 2023年3月2日
在6.2 compat层中,支持指向结束标记的书签。这是为了与WP 6.2中的Core对等,请参见WordPress/WordPress-develop#4115和[55407兰特](https://core.trac.wordpress.org/changeset/55407).请注意,这些更改已在中回传到GB的6.**3**compat层#48378然而,由于更改将在WP 6.2中进行,我们还必须将其应用于GB的6.2兼容层。
免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
还没有
项目
还没有
4名参与者