跳到内容
新问题

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

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

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

修复损坏的导航保存 #23559

已合并
将1个提交合并到 2020年6月29日
已合并

修复损坏的导航保存 #23559

将1个提交合并到 2020年6月29日

对话

阿达齐尔
复制链接
贡献者

@阿达齐尔 阿达齐尔 评论2020年6月29日

描述

此PR修复了附加新菜单项后保存导航的问题。问题的根本原因是创建MissingMenuItems由于编辑导航该包使用核心挂钩,并陷入缓存效果的复杂链中。具体来说,渲染<EditorProvider onChange={回调}缓存回调在第一次呈现时传递,并将继续调用该初始版本,即使在后续呈现时传递更新的回调。这反过来意味着创建MissingMenuItems总是被第一个版本的导航栏调用。

虽然重构代码是可能的,但它也很复杂。本公关中提出的修复方案并没有直接解决问题,但它通过以下行为解决了问题序列化处理更具内部一致性&每个等待串行执行的回调都会得到一个新选择的post。这与之前完全相同,只是队列中的第一个回调也将受益于此行为。

这是如何测试的?

  1. 启用实验导航屏幕并转到那里
  2. 创建新菜单或编辑现有菜单
  3. 添加一些菜单项
  4. 保存
  5. 确认它有效

变更类型

错误修复(修复问题的非破坏性更改)

检查清单:

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

@github-操作
复制链接

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

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

总大小:1.13 MB

文件名 大小 更改
构建/编辑导航/index.js 9.87千字节 +3亿(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/index.js 130千字节 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亿

压缩尺寸作用

@德拉加内斯库
复制链接
贡献者

@阿达齐尔“而不是解开”是什么意思。实体中有漏洞吗?

@阿达齐尔
复制链接
贡献者 作者

阿达齐尔 评论2020年6月29日

@德拉加内斯库不,实体中没有错误,我只是用更多信息更新了描述

复制链接
贡献者

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

选择隐藏此评论的原因

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

本地和LGTM测试🚢

@德拉加内斯库 德拉加内斯库合并提交46安04进入之内 主人 2020年6月29日
导航编辑器 自动化将此从PR进行中多恩 2020年6月29日
@德拉加内斯库 德拉加内斯库删除了 修复/中断导航-保存 分支 2020年6月29日17:36
@github-操作 github-操作 机器人程序将此添加到古腾堡8.5里程碑2020年6月29日
免费注册 在GitHub上加入此对话.已经有账户了吗?登录以发表评论
标签
还没有
项目
没有打开的项目
导航编辑器
  
多恩
开发

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

还没有

2名参与者