跳到内容
新问题

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

注册GitHub

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

已经在GitHub上了？登录到您的帐户
跳到底部

在实验导航屏幕上隐藏导航块工具栏 #23021

已合并
将8个提交合并到从 2020年6月30日
已合并

在实验导航屏幕上隐藏导航块工具栏 #23021

将8个提交合并到从 2020年6月30日

对话

阿达齐尔
复制链接
贡献者

@阿达齐尔 阿达齐尔 评论2020年6月9日 •
已编辑

描述

解决方案#21310通过隐藏导航块工具栏，因为它不再包含任何有用的按钮：

Zrzut ekranu邮编：2020-06-9 o 13 02 50

这是如何测试的？

  1. 在古腾堡>实验中启用导航实验
  2. 转到古腾堡>导航（测试版）
  3. 确认工具栏不包含对齐、导航器、颜色或空设置菜单

变更类型

非破坏性变化

检查清单：

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

@阿达齐尔 阿达齐尔 用力推动的这个 更新/删除不相关的工具栏项 分支从 b042327号 到 9293952 比较 2020年6月9日11:04
@github-操作
复制链接

github-操作 机器人程序 评论2020年6月9日 •
已编辑

尺寸变化：-6亿（0%）

总大小：1.13 MB

文件名 大小 更改
build/block-library/index.js 130千字节 -17亿（0%）
构建/编辑导航/index.js 9.88千字节 +11亿（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.39千字节 0亿
构建/块目录/样式-rtl.css 941亿 0亿
构建/块目录/样式.css 942亿 0亿
build/block-editor/index.js 109千字节 0亿
构建/块编辑器/style-rtl.css 10.7千字节 0亿
构建/块编辑器/样式.css 10.7千字节 0亿
build/block-library/editor-rtl.css 7.6千字节 0亿
build/block-library/editor.css 7.6千字节 0亿
build/block-library/style-rtl.css 8.04千字节 0亿
build/block-library/style.css 8.04千字节 0亿
构建/块库/theme-rtl.css 730亿 0亿
构建/块库/theme.css 732亿 0亿
构建/块序列化默认解析器/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亿
构建/组件/样式.css 15.9千字节 0亿
构建/合成/索引.js 9.64千字节 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亿
build/dom-ready/index.js版本 569亿 0亿
build/dom/index.js版本 3.19千字节 0亿
构建/编辑导航/样式-rtl.css 1.02千字节 0亿
构建/编辑导航/样式.css 1.02千字节 0亿
build/edit-post/index.js 303千字节 0亿
构建/编辑-发布/样式-rtl.css 5.51千字节 0亿
构建/编辑-发布/样式.css 5.5千字节 0亿
build/edit-site/index.js 16.6千字节 0亿
构建/编辑site/style-rtl.css 3.03千字节 0亿
构建/编辑site/style.css 3.03千字节 0亿
build/edit-widgets/index.js 9.32千字节 0亿
build/edit-widgets/style-rtl.css 2.42千字节 0亿
构建/编辑小部件/style.css 2.42千字节 0亿
build/editor/editor-styles-rtl.css 537亿 0亿
构建/编辑器/编辑器样式.css 539亿 0亿
build/editor/index.js 44.8千字节 0亿
构建/编辑器/样式-rtl.css 3.86千字节 0亿
构建/编辑器/样式.css 3.86千字节 0亿
构建/元素/索引js 4.65千字节 0亿
构建/转义-html/index.js 733亿 0亿
build/format-library/index.js 7.72千字节 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亿
build/i18n/index.js版本 3.56千字节 0亿
build/浅等于/index.js 711亿 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 450亿 0亿
build/list-rusable-blocks/style.css 451亿 0亿
build/media-utils/index.js 5.29千字节 0亿
build/notices/index.js 1.79千字节 0亿
构建/nux/index.js 3.4千字节 0亿
build/nux/style-rtl.css 663亿 0亿
build/nux/style.css 660亿 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.7千字节 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月9日
查看已审阅的更改
packages/block-editor/src/components/block-toolbar/index.js 过时的
@@-57,6+58,8@@导出默认函数BlockToolbar（{hideDragHandle}）{
getBlockType（getBlockName（selectedBlockClientId）），
hasFixedToolbar:getSettings（）.hasFixed工具栏，
rootClientId:blockRootClientId，
isNavigationManagementScreen:getSettings（）
.__experimentalNavigation屏幕，
复制链接
贡献者

选择隐藏此评论的原因

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

我认为“区块编辑器”模型不应该知道导航屏幕的存在。

复制链接
贡献者

选择隐藏此评论的原因

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

我们以前有一个名为uiParts或类似的配置，我认为它更“通用”，我们可能会想把它带回来。

复制链接
贡献者 作者

选择隐藏此评论的原因

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

@你知道riad好主意-我刚更新了代码

@阿达齐尔 阿达齐尔 用力推动的这个 更新/删除-irelevant-toolbar-items 分支从 9293952 到 d20c4a9 比较 2020年6月10日13:26
你知道riad
你知道riad 检验过的 2020年6月10日
查看已审阅的更改
packages/edit-navigation/src/index.js 过时的
@@-58,6+58,10@@导出函数初始化（id，设置）{
}
设置__experimentalFetchLinkSuggestions=获取链接建议；
设置__experimentalNavigationScreen=true；
复制链接
贡献者

选择隐藏此评论的原因

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

我刚刚注意到我们已经有了，我想是类似的吧？

复制链接
贡献者 作者

选择隐藏此评论的原因

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

鹰眼！尼斯：-）这个公共关系删除了它的所有用途，所以它不再发挥任何作用-我应该删除它，我刚才就这么做了。

@德拉加内斯库 德拉加内斯库已将此添加到PR进行中在里面导航编辑器 2020年6月10日
塔尔丹
塔尔丹 检验过的 2020年6月26日
查看已审阅的更改
复制链接
贡献者

@塔尔丹 塔尔丹 留下了评论

选择隐藏此评论的原因

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

谢谢你的努力，亚当！

与您在公关描述中提到的类似，我想知道在这一更改之后，导航块可以以任何方式进行选择的价值。

这也使得这个PR变得无关紧要，所以我们可能想删除这些更改-#22737.

包/block-library/src/navigation/edit.js 过时的
<工具栏组>{navigatorToolbarButton}
) }
<片段>
{显示格式按钮工具栏&&(
复制链接
贡献者

选择隐藏此评论的原因

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

实现此目的的另一种方法是过滤块编辑仅在中编辑导航然后注入另一种道具禁用工具栏选项:
https://developer.wordpress.org/block-editor/developers/filters/block-filters/#editor-块编辑

无论哪种方式我都可以，但我想我应该提一下这个想法。

包/block-library/src/navigation/edit.js 过时的
select（'核心/块编辑器'）.getSettings（）
.__实验导航屏幕
select（'核心/块编辑器'）.getSettings（）.__experimentalUi部件
.导航块工具栏格式
复制链接
贡献者

选择隐藏此评论的原因

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

我想用这个词格式格式通常指的是粗体或斜体，而不是工具栏的其他部分，因此可能会造成混淆。

也许 吧导航块工具栏控件?

话虽如此，由于! 已选择根块条件（在packages/edit-navigation/src/components/navigage-editor/block-editor-area.js中），所以这里的设置可能根本不需要。

包/edit-navigation/src/components/navigation-editor/block-editor-area.js 显示已解决 隐藏已解决
packages/block-editor/src/components/block-toolbar/index.js 过时的
@@-129,7+132,11@@导出默认函数BlockToolbar（{hideDragHandle}）{
onDragStart={onDraggableStart}
onDragEnd={onDraggableEnd}
>
<BlockSwitcher clientIds={blockClientIds}/>
{shouldShowBlockSwitcher&&(
复制链接
贡献者

选择隐藏此评论的原因

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

我想知道是否需要这个设置，因为! 已选择根块此PR中的条件（在packages/edit-navigation/src/components/navigage-editor/block-editor-area.js中）。

对于导航链接，它充当块图标，但作用不大且无害。

我个人倾向于尽可能缩小这个PR并放弃它。

如果切换器菜单的某些部分没有用处（例如块样式），最好添加一种方法来禁用编辑导航的这些功能。

@阿达齐尔 阿达齐尔 用力推动的这个 更新/删除-irelevant-toolbar-items 分支从 2cfbcf1 到 19第2页4 比较 2020年6月29日13:46
@阿达齐尔
复制链接
贡献者 作者

@塔尔丹很好，我已经回复了您的反馈，此公关已准备好重新审查：-）

@阿达齐尔 阿达齐尔更改了标题从实验导航屏幕中删除无关的块工具栏项 在实验导航屏幕上隐藏导航块工具栏 2020年6月30日
德拉加内斯库
德拉加内斯库 批准了这些更改 2020年6月30日
查看已审阅的更改
复制链接
贡献者

@德拉加内斯库 德拉加内斯库 留下了评论

选择隐藏此评论的原因

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

在本地进行了测试，工具栏不见了！

复写的副本@肖南德鲁注意这一变化。因此，现在当我们打开屏幕时，导航块顶部不再有按钮。不过，我们将来如何更改导航布局？还有一个关于在本页中设置检查器的提示。

@德拉加内斯库 德拉加内斯库合并提交8a07cdb进入之内 主人 2020年6月30日
导航编辑器 自动化将此从PR进行中到多恩 2020年6月30日
@德拉加内斯库 德拉加内斯库删除了 更新/删除-irelevant-toolbar-items 分支 2020年6月30日11:44
@github-操作 github-操作 机器人程序将此添加到古腾堡8.5里程碑2020年6月30日
关闭
@告诉机器 告诉机器提到这个拉请求 2020年8月18日
关闭
关闭
免费注册 在GitHub上加入此对话.已经有账户了吗？登录以发表评论
审阅者

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

@塔尔丹 塔尔丹 塔尔丹留下了评论

@德拉加内斯库 德拉加内斯库 德拉加内斯库批准了这些修改

@吵闹声 吵闹声 等待Noissocks要求的审查

@阿吉博拉 阿吉博拉 等待ajitbohra要求的审查

@埃拉特里克斯 埃拉特里克斯 等待ellatrix的审查请求

@Soean公司 大豆 等待Soean的审查请求

@告诉机器 告诉机器 等待通知机器的审查请求

受让人

@阿达齐尔 阿达齐尔

标签
还没有
项目
没有打开的项目
导航编辑器
  
多恩
里程碑
古腾堡8.5
开发

成功合并此拉取请求可能会解决这些问题。

还没有
4名参与者
@阿达齐尔 @德拉加内斯库 @你知道riad @塔尔丹