跳到内容
新问题

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

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

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

将可重复使用的块选项卡添加到插入器。 #23296

已合并
将4个提交合并到 2020年7月3日

对话

泽布兰斯坦菲尔
复制链接
成员

@泽布兰斯坦菲尔 泽布兰斯坦菲尔 评论2020年6月19日

描述

关闭 #22860.
关闭 #23253

此PR将可重用块从插入器中标准块选项卡底部隐藏的类别移动到其自己的选项卡。

目前,它使用块选项卡的样式来显示可重用块。但我认为以与模式相同的方式呈现它们可能更有意义。让我知道你的想法#22860个.

这是如何测试的?

我测试了该选项卡是否显示了可重用块,插入可重用块是否有效,以及可重用类别是否从标准块选项卡中删除。

屏幕截图

形象

检查清单:

  • 我的代码已经过测试。
  • 我的代码遵循WordPress代码风格。
  • 我的代码遵循可访问性标准。
  • 我的代码有适当的内联文档。
  • 如果合适的话,我已经包括了开发人员文档。
  • 我已经更新了所有受此PR中任何重构/重命名影响的React Native文件。

@泽布兰斯坦菲尔 泽布兰斯坦菲尔补充[类型]增强功能 改进建议。 [功能]插入器 在编辑界面中使用+按钮插入块的主要方式 [功能]同步模式 与同步模式(以前的可重用块)相关 需要技术反馈 需要从开发人员的角度进行测试。 需要辅助功能反馈 需要可访问性的输入标签2020年6月19日
@github-操作
复制链接

github-操作 机器人程序 评论2020年6月19日

尺寸变化:+181亿(0%)

总大小:1.13 MB

文件名 大小 更改
build/block-editor/index.js 109千字节 +169亿(0%)
build/block-editor/style-rtl.css 10.7千字节 +6亿(0%)
构建/块编辑器/样式.css 10.7千字节 +6亿(0%)
ℹ️查看未更改
文件名 大小 更改
构建/a11y/index.js 1.14千字节 0亿
构建/注释/index.js 3.62千字节 0亿
构建/api-fetch/index.js 3.4千字节 0亿
构建/auto/index.js 2.82千字节 0亿
构建/blob/index.js 620亿 0亿
build/block-directory/index.js 7.16千字节 0亿
构建/块目录/样式-rtl.css 952亿 0亿
构建/块目录/样式.css 951亿 0亿
build/block-library/editor-rtl.css 7.63千字节 0亿
build/block-library/editor.css 7.64千字节 0亿
build/block-library/index.js 130千字节 0亿
build/block-library/style-rtl.css 7.79千字节 0亿
build/block-library/style.css 7.79千字节 0亿
构建/块-库/theme-rtl.css 730亿 0亿
构建/块-库/theme.css 732亿 0亿
build/block-serialization-default-parser/index.js 1.88千字节 0亿
build/block-serialization-specifi-parser/index.js 3.1千字节 0亿
build/blocks/index.js 48.2千字节 0亿
build/components/index.js 198千字节 0亿
build/components/style-rtl.css 15.9千字节 0亿
build/components/style.css 15.8千字节 0亿
构建/合成/索引.js 9.65千字节 0亿
构建/core-data/index.js 11.4千字节 0亿
构建/数据控制/索引.js 1.29千字节 0亿
build/data/index.js 8.44千字节 0亿
构建/日期/索引.js 5.47千字节 0亿
build/deprecated/index.js 772亿 0亿
构建/dom-ready/index.js 569亿 0亿
build/dom/index.js版本 3.19千字节 0亿
构建/编辑导航/index.js 9.97千字节 0亿
构建/编辑导航/样式-rtl.css 1.02千字节 0亿
构建/编辑导航/样式.css 1.02千字节 0亿
build/edit-post/index.js 304千字节 0亿
构建/编辑-发布/样式-rtl.css 5.57千字节 0亿
构建/编辑-发布/样式.css 5.56千字节 0亿
build/edit-site/index.js 16.6千字节 0亿
构建/编辑site/style-rtl.css 3.02千字节 0亿
构建/编辑site/style.css 3.02千字节 0亿
build/edit-widgets/index.js 9.33千巴 0亿
build/edit-widgets/style-rtl.css 2.45千字节 0亿
build/edit-widgets/style.css 2.45千字节 0亿
build/editor/editor-styles-rtl.css 537亿 0亿
构建/编辑器/编辑器样式.css 539亿 0亿
build/editor/index.js 44.8千字节 0亿
构建/编辑器/样式-rtl.css 3.82千字节 0亿
构建/编辑器/样式.css 3.82千字节 0亿
build/element/index.js 4.65千字节 0亿
构建/转义-html/index.js 733亿 0亿
构建/格式化库/index.js 7.73千字节 0亿
build/format-library/style-rtl.css 547亿 0亿
构建/格式库/样式.css 548亿 0亿
build/hooks/index.js 2.13千字节 0亿
build/html-entities/index.js 622亿 0亿
构建/i18n/index.js 3.56千字节 0亿
build/is-shallow-equal/index.js 710亿 0亿
构建/键盘快捷方式/index.js 2.51千字节 0亿
build/keycodes/index.js 1.94千字节 0亿
build/list-reasable-blocks/index.js 3.12千字节 0亿
构建/list-rusable-blocks/style-rtl.css 476亿 0亿
build/list-rusable-blocks/style.css 476亿 0亿
build/media-utils/index.js 5.3千字节 0亿
build/notices/index.js 1.79千字节 0亿
构建/nux/index.js 3.4千字节 0亿
build/nux/style-rtl.css 671亿 0亿
build/nux/style.css 668亿 0亿
build/plugins/index.js 2.56千字节 0亿
build/primitives/index.js 1.5千字节 0亿
构建/priority-queue/index.js 788亿 0亿
build/redux-routine/index.js 2.85千字节 0亿
构建/rich-text/index.js 14千字节 0亿
build/server-side-render/index.js 2.68千字节 0亿
build/shortcode/index.js 1.69千巴 0亿
构建/token-list/index.js 1.28千字节 0亿
构建/url/index.js 4.06千字节 0亿
build/viewport/index.js 1.85千字节 0亿
build/warning/index.js 1.14千字节 0亿
构建/wordcount/index.js 1.17千字节 0亿

压缩尺寸作用

@你知道riad 你知道riad补充这个需要设计反馈 需要一般的设计反馈。标签2020年6月19日
@迈克尔·阿雷斯塔德
复制链接
贡献者

这很有效。从设计角度来看,我唯一的建议是更改标签,因为@米提亚斯建议改为“已保存”

@泽布兰斯坦菲尔 泽布蓝斯坦菲尔补充这个需要副本审阅 需要审查用户界面副本(语言、短语)标签2020年6月20日
@泽布兰斯坦菲尔
复制链接
成员 作者

这有意义吗?在一个地方称它们为“保存的”,而在其他地方称它们是“可重用的”,这似乎很奇怪,也不一致。

@米歇尔·韦伯
复制链接

我倾向于坚持使用“可重用”来保持一致性——可重用块的存在/想法仍然是新的,所以我希望人们有信心通过寻找我们在其他地方使用的“可重用的”语言来找到他们需要的东西。

@米提亚斯
复制链接
成员

我认为我们应该重命名为已保存如果我们把它提升为自己的分组(我喜欢),那么就可以实现全面性。我看到了可重复使用的在一些语言中存在上下文不足(可重用块比可重用块好得多)和翻译缺陷。Saved本身在Patterns和Blocks旁边就好一点了。

这就是说,我们应该分别研究重命名,并查看显示动作和图标的不同上下文。

类别,
收藏,
过滤器值
).filter(({category})=>类别===“可重用”);
复制链接
成员

选择隐藏此评论的原因

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

对于这个公共关系来说不是这样,但我认为我们应该放弃“可重用”作为一个类别,而是允许选项卡显示其他类别。这将允许站点对可重用块进行分类(在管理自定义帖子类型时使用常规类别界面),并将此选项卡中列出的类别作为常规块类别。

复制链接
成员 作者

选择隐藏此评论的原因

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

同意。

@恩里克桑切斯
复制链接
贡献者

在Safari+VoiceOver上测试了此PR。选项卡的名称和选项卡的数量(即3个选项卡中的3个)以及每个选项卡下的块数(Patterns除外)都像以前一样正确地公布了。

从可访问性的角度来看,这对我来说很好👍

@泽布兰斯坦菲尔 泽布兰斯坦菲尔远离的需要辅助功能反馈 需要可访问性的输入 需要副本审阅 需要审查用户界面副本(语言、短语)标签2020年6月24日
@泽布兰斯坦菲尔
复制链接
成员 作者

之后#22789合并后,我需要重新设置此基础。我做了,现在可重用块选项卡没有出现。我修好后会告诉你的。

@泽布兰斯坦菲尔 泽布兰斯坦菲尔 用力推动的这个 添加/插入器可重复使用的块选项卡 分支2次,最近一次 a6a2134号 8个92分贝 比较 2020年6月25日03:22
@泽布兰斯坦菲尔 泽布蓝斯坦菲尔 用力推动的这个 添加/插入器-可重用块标签 分支2次,最近一次 公元前9d192年 6cb6761号 比较 2020年6月29日23:58
@泽布兰斯坦菲尔
复制链接
成员 作者

好的,我花了一整天的时间来修复端到端测试。我只做了一次失败的测试,我不知道为什么它会失败:

失败包/e2e-tests/specs/editor/plugins/cpt-locking.test.js(43.064s)cpt锁定›template_lock all›可以在内部块中使用全局插入器expect(received).非.toBeNull()收到:空120 |等待按KeyTimes('Tab',2);121|等待页面.keyboard.press('Enter');>122|expect(等待页面.$('.wp-block-gallery')).not.toBeNull();| 			                                                  ^123 | 		} );124 | 	} );125 |位于Object<匿名>(specs/editor/plugins/cpt-locking.test.js:122:54)在运行Microtasks时(<匿名>)

似乎添加了Reusable块选项卡(导致插入器使用选项卡,从而导致额外的选项卡但如果是这种情况(可能不是这样),我不知道是什么原因导致Reusable block选项卡出现。我需要帮助来解决这个问题。

@贾斯穆森
复制链接
贡献者

我看了一下,测试通过了。

我甚至尝试了这个互动测试:

npm运行test-e2e包/e2e-tests/specs/editor/plugins/cpt-locking.test.js--puppeter-interactive

这个失败的测试在视觉上似乎也能达到预期效果。以下是测试位:

等待页面。单击(“.edit-post-header-toolbar__inserter-toggle”);等待页面.type('.block-editor-inserter__search-input','画廊');等待按KeyTimes(“制表符”,2);等待页面.keyboard.press(“回车”);expect(等待页面.$('.wp-block-gallery')).not.toBeNull();

因此,单击插入器,搜索“gallery”,按tab键两次,您应该已经插入了一个库块。这就是有趣的地方,因为无论配置如何,我总是需要按3次tab键。这是没有可重用块的分支:

没有可重用的

这是您的分支,带有可重用块:

画廊

以下是具有可重用块的主分支:

可重复使用的master

在这三种情况下,您按了三次tab键。这就引出了问题:

  • 为什么测试要通过master?
  • 为什么我在这个分支上的测试在本地通过了,但在这里却失败了?

@你知道riad-豪尔赫写下了这些测试,但被淘汰了。你有什么见解吗?


assertNoResultsMessageNotToBePresent(容器);
} );
} );
复制链接
贡献者

选择隐藏此评论的原因

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

我不太确定这个单元测试对我个人来说是否很有价值,但你可以打电话给我。

复制链接
成员 作者

选择隐藏此评论的原因

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

我不是从零开始创建这些测试的。它们最初位于标准块选项卡的单元测试文件中。由于可重用块现在位于它们自己的选项卡中,所以我只是将它们移动了一下。

@你知道riad
复制链接
贡献者

我认为测试失败的原因是,如果您有可重用的块,那么在插入器上会有一个额外的制表位。
理想情况下,在end2end测试开始时删除可重用块(如post),创建这些块的end2end-测试应该在完成时清理它们。

复制链接
贡献者

@你知道riad 你知道riad 留下了评论

选择隐藏此评论的原因

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

很酷的东西,这看起来不错,我留下了一些关于e2e测试错误的小评论和建议。

@你知道riad
复制链接
贡献者

小提示:与其修改提交(尽管有时是必要的),不如继续在PR中推送新的提交。这使得关注公共关系的人更容易了解最新提交中发生了什么变化。

@泽布兰斯坦菲尔
复制链接
成员 作者

我没有意识到可重用的块在测试中是持久的。有没有一种简单的方法可以在测试结束时删除所有可重用的块?或者您必须“手动”执行删除吗(通过打开每个可重用块上的更多菜单并单击“从可重用块中删除”选项)。

@你知道riad
复制链接
贡献者

@泽布兰斯坦菲尔有一个现有垃圾站中的函数setup-test-framework.js,我们应该对可重用块执行类似的操作。(我们可能会把它变成一个实用程序)

@你知道riad 你知道riad 用力推动的这个 添加/插入器-可重用块标签 分支 1406d6b型 c4b6798型 比较 2020年7月3日12:22
@你知道riad 你知道riad 用力推动的这个 添加/插入器-可重用块标签 分支 c4b6798型 父亲3772 比较 2020年7月3日12:37
复制链接
贡献者

@你知道吗里亚德 你知道riad 留下了评论

选择隐藏此评论的原因

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

让我们为5.5 beta 1做好准备。

💯 这里的荣誉@泽布兰斯坦菲尔

@你知道riad 你知道riad合并提交9电子0c923进入之内 主人 2020年7月3日
WordPress 5.5必须具备 自动化将此从需要审查多恩 2020年7月3日
@你知道riad 你知道riad删除了 添加/插入器-可重用块标签 分支 2020年7月3日13:05
@github-操作 github-操作 机器人程序将此添加到古腾堡8.5里程碑2020年7月3日
@贾斯穆森
复制链接
贡献者

👌

@泽布兰斯坦菲尔
复制链接
成员 作者

感谢您修复测试,@你知道riad😄

免费注册 在GitHub上加入此对话.已经有帐户了吗?登录以发表评论
标签
[功能]插入器 在编辑界面中使用+按钮插入块的主要方式 [功能]同步模式 与同步模式(以前的可重用块)相关 需要技术反馈 需要从开发人员的角度进行测试。 [类型]增强功能 改进建议。
项目
没有打开的项目
7名参与者