Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@2betop
Copy link
Member

@2betop 2betop commented Jul 19, 2024

What

  1. 优化大纲,使得更紧凑,同时高亮 hover 区域优化
  2. 优化拖拽,改成通过功能键切换拖拽容器,或者拖到左侧大纲切换。
  3. 优化拖拽过程中的大纲视图,区域更好拖入

Why

How

@2betop 2betop marked this pull request as draft July 19, 2024 04:36
@github-actions
Copy link

👍 Thanks for this!
🏷 I have applied any labels matching special text in your issue.

Please review the labels and make any necessary changes.

@github-actions github-actions bot added the chore label Jul 19, 2024
@2betop 2betop force-pushed the chore-editor-718 branch from 162e0e6 to 53f6ab0 Compare July 25, 2024 12:56
@2betop 2betop force-pushed the chore-editor-718 branch from 53f6ab0 to c8fa2d9 Compare July 25, 2024 13:15
@2betop 2betop marked this pull request as ready for review July 25, 2024 13:15
@hsm-lv hsm-lv merged commit c785983 into baidu:master Aug 7, 2024
@2betop 2betop deleted the chore-editor-718 branch August 7, 2024 03:18
@ljmCoder
Copy link

@2betop 能麻烦请问下,目前最新的master代码, 在将某个组件拖拽到另外一个容器时, 拖不进去, 有没有可能是这块代码的问题?

@2betop
Copy link
Member Author

2betop commented Aug 28, 2024

@2betop 能麻烦请问下,目前最新的master代码, 在将某个组件拖拽到另外一个容器时, 拖不进去, 有没有可能是这块代码的问题?

目前直接拖放不会切换容器了,需要结合功能键或者先在左侧拖入指定容器后再控制拖入的位置

@ljmCoder
Copy link

@2betop 能麻烦请问下,目前最新的master代码, 在将某个组件拖拽到另外一个容器时, 拖不进去, 有没有可能是这块代码的问题?

目前直接拖放不会切换容器了,需要结合功能键或者先在左侧拖入指定容器后再控制拖入的位置

image
如图。最新的master分支代码。我将第一个flex容器里面的title组件拖拽到左侧面板中的第三个容器。 释放鼠标没有反应, 组件并没有移动过去

@ljmCoder
Copy link

@2betop 能麻烦请问下,目前最新的master代码, 在将某个组件拖拽到另外一个容器时, 拖不进去, 有没有可能是这块代码的问题?

目前直接拖放不会切换容器了,需要结合功能键或者先在左侧拖入指定容器后再控制拖入的位置

image 如图。最新的master分支代码。我将第一个flex容器里面的title组件拖拽到左侧面板中的第三个容器。 释放鼠标没有反应, 组件并没有移动过去

@2betop

@2betop
Copy link
Member Author

2betop commented Aug 28, 2024

嗯,左侧那个目前只用于切容器,不能直接释放,这个我去优化一下。目前需要回到右侧预览区释放才有效果

@ljmCoder
Copy link

嗯,左侧那个目前只用于切容器,不能直接释放,这个我去优化一下。目前需要回到右侧预览区释放才有效果

好的, 还发现了一个问题, 可以一起修复下, 还是上面的场景, title被从flex的A容器拖到B容器时, 会导致A容器的body字段丢失, 会导致显示有问题(理论上A、C应该显示一致)
image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants