跳到内容
新问题

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

注册GitHub

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

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

将块可拖动滚动行为提取到React挂钩 #23444

已合并
将4个提交合并到从 2020年6月26日
已合并

将区块可拖动滚动行为提取到React钩子中 #23444

将4个提交合并到从 2020年6月26日

对话

塔尔丹
复制链接
贡献者

@塔尔丹 塔尔丹 评论2020年6月25日 •
已编辑

描述

如中所述https://github.com/WordPress/gutenberg/pull/23082/files#r444178859，拖动时的新滚动行为可以非常整齐地封装到React挂钩中。

这个PR将代码移动到一个钩子中，几乎是一字不差的，只需重命名和更新一些注释即可满足编码标准。

这是如何测试的？

  • 块拖动/滚动行为应与中的相同主人.

变更类型

非破坏性代码质量更改

检查清单：

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

@塔尔丹 塔尔丹补充[类型]代码质量 与代码质量相关的问题或PR [功能]拖放 使用块时的拖放功能标签2020年6月25日
@塔尔丹 塔尔丹自我分配的2020年6月25日
@github-操作
复制链接

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

尺寸变化：+61亿（0%）

总大小：1.13 MB

文件名 大小 更改
build/block-editor/index.js 109千字节 +61 B（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.37千字节 0亿
构建/块目录/样式-rtl.css 941亿 0亿
构建/块目录/样式.css 942亿 0亿
构建/块编辑器/style-rtl.css 10.7千字节 0亿
构建/块编辑器/样式.css 10.7千字节 0亿
build/block-library/editor-rtl.css 7.59千字节 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.62千字节 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亿
构建/编辑导航/index.js 9.87千字节 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.02千字节 0亿
构建/编辑site/style.css 3.02千字节 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.83千字节 0亿
构建/编辑器/样式.css 3.83千字节 0亿
构建/元素/索引js 4.65千字节 0亿
构建/转义-html/index.js 733亿 0亿
build/format-library/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亿
build/i18n/index.js版本 3.56千字节 0亿
build/浅等于/index.js 710亿 0亿
构建/键盘快捷方式/index.js 2.51千字节 0亿
build/keycodes/index.js 1.94千字节 0亿
build/list-reasable-blocks/index.js 3.13千字节 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 789亿 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月25日
关闭
6项任务
你知道riad
你知道riad 检验过的 2020年6月25日
查看已审阅的更改
packages/block-editor/src/components/block-draggable/index.js 过时的
const scrollParentY=useRef（空）；

const scrollEditorInterval=useRef（null）；
const blockElementId=`block-${clientIds[0]}`；
复制链接
贡献者

选择隐藏此评论的原因

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

我认为现在我们有了一种更好的方法，可以使用“BlockNodes”上下文从树中获取块节点，这意味着我们可以完全避免这种争论。

复制链接
贡献者 作者

选择隐藏此评论的原因

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

哦，是的，我记得看到过。我已经更改了代码以使用它。

我还注意到，钩子中的代码对任何类型的拖放事件都是不可知的，所以我让钩子接受dragElement（拖动元素）和块可拖动现在从中获取块的元素块节点要传递到钩子中的上下文。

复制链接
贡献者 作者

选择隐藏此评论的原因

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

我在手动测试时没有发现任何问题，但这种更改似乎导致了端到端测试的一些问题。

我明天会修好它。

你知道riad
你知道riad 批准了这些更改 2020年6月25日
查看已审阅的更改
复制链接
贡献者

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

选择隐藏此评论的原因

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

我喜欢这种重构，我也想知道我们是否可以从其他PR中删除“key”和对BlockPopover的更改。

这是被引用的2020年6月25日
已合并
已合并
@帕尔约阿希姆
复制链接
贡献者

帕尔约阿希姆 评论2020年6月25日 •
已编辑

这里可以使用相同的程序吗？
尝试：拖动时不显示克隆。
#23024

丹，谢谢你的努力！

@塔尔丹 塔尔丹 用力推动的这个 refactor/block-drag-scroll-behavior-in-to-hook 分支从 fd06641型 到 5c79亿b43 比较 2020年6月26日02:49
@塔尔丹
复制链接
贡献者 作者

除了PHP测试似乎一夜之间就开始失败外，测试现在已经通过了。我看不到任何可能导致这种情况的合并公关，所以很可能是环境问题。

使用管理员权限合并，因为它不包含PHP更改。

@塔尔丹 塔尔丹合并提交d074d26进入之内 主人 2020年6月26日
@塔尔丹 塔尔丹删除了 refactor/block-drag-scroll-behavior-in-to-hook 分支 2020年6月26日03:13
@github-操作 github-操作 机器人程序将此添加到古腾堡8.5里程碑2020年6月26日
@塔尔丹 塔尔丹提到这个拉请求 2020年6月26日
已合并
6项任务
免费注册 在GitHub上加入此对话.已经有账户了吗？登录以发表评论
审阅者

@你知道riad 你知道riad 你知道riad批准了这些更改

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

@重力旋翼 重力轨道 等待重力尾的审查请求

受让人

@塔尔丹 塔尔丹

标签
[功能]拖放 使用块时的拖放功能 [类型]代码质量 与代码质量相关的问题或PR
项目
还没有
里程碑
古腾堡8.5
开发

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

还没有
3名参与者
@塔尔丹 @帕尔约阿希姆 @你知道riad