问题 #5175💬 已回答提出于 2023年6月10日作者 krishnaeverestengineering0 个反应

移动方法没有按预期工作

快速解答作者 krishnaeverestengineering

解决方法：https://github.com/GrapesJS/grapesjs/pull/5176

问题

GrapesJS 版本 [X] 我确认使用最新版本 GrapesJS 你用的是什么浏览器？铬可复制演示链接 https://codesandbox.io/s/adoring-margulis-873kdv 描述一下那个虫子背景移动组件的方法没有按预期工作。这段代码没有按预期工作。这段代码应该可以正常工作。当 sameParent 是 false 且目标索引是 1 时，比如它不会移动到 0 对 1 的位置行为准则 [X] 我同意遵守本项目的行为准则

回答（4）

krishnaeverestengineering• 2023年6月12日

解决方法：https://github.com/GrapesJS/grapesjs/pull/5176

artf• 2023年6月15日

谢谢@krishnaeverestengineering，但这实际上是将组件移入同父节点时的预期行为。这是因为“at”还应考虑组件本身的索引。

- 指数：0
{ 组件1 }
- 指数：1
{ 组件2 }
- 指数：2

所以，如果你尝试将“组件1”移到索引0或1，什么都不会发生。如果你需要把它移到“组件2”以下，你必须标注索引2

krishnaeverestengineering• 2023年6月15日

谢谢@krishnaeverestengineering，但这实际上是将组件移入同一父节点时的预期行为。这是因为“at”还应考虑组件本身的索引。
- 指数：0
{ Component 1 }
- 指数：1
{ Component 2 }
- 指数：2
所以，如果你尝试将“组件1”移到索引0或1，什么都不会发生。如果你需要把它移到“组件2”以下，你必须标注索引2

但我不太理解它的使用场景。通常在最后版本中，'at '应该算是索引。那怎么把组件从0移到索引1呢？

GJS Helper• 2026年5月17日

你已经发现了GrapesJS Component.prototype.move方法中的一个关键漏洞。当前的实现存在两个主要问题，导致组件放置错误，并且在某些情况下，组件根本无法移动，尤其是在同一父节点内移动时。根本原因分析有缺陷的sameIndex检查：原始的sameIndex计算是index === at || index === at - 1。如果一个组件当前的索引比目标at索引少一个，则错误地标记为“同一索引”。例如，如果某个组件位于索引0，你试图将其移到索引1，index === at - 1（0 === 1 - 1）会被评估为true。这会导致sameIndex为true，如...

在 GitHub 上查看完整讨论 →