Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Outline drag function is abnormal #10828

Closed
TCOTC opened this issue Apr 2, 2024 · 32 comments
Closed

Outline drag function is abnormal #10828

TCOTC opened this issue Apr 2, 2024 · 32 comments
Assignees
Labels
Milestone

Comments

@TCOTC
Copy link
Contributor

TCOTC commented Apr 2, 2024

大纲拖拽功能异常

  1. 要拖拽到同级大纲下才能拖拽成功
  2. 拖拽后如果不改变位置会置顶

大纲2

大纲

Version environment

- Version: 3.0.6
- Operating System: Win10
- Browser (if used):
@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

关联 #10826

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

单纯点击时不应该拖拽(像在文档树拖拽文档那种)

image

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

在特殊情况下大纲能被带走(在文档树拖拽文档时则不会发生这种情况):

大纲3

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

无法正确撤回:

大纲5

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

大纲层级异常降低,并且撤回后的层级也是错的:

大纲4

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

层级6拖拽到层级3,会变成层级4

@TCOTC TCOTC changed the title 大纲拖拽排序异常 大纲拖拽功能异常 Apr 2, 2024
@88250
Copy link
Member

88250 commented Apr 2, 2024

v3.0.6 大纲拖曳列表标题吞块 - 链滴

5ejto-7q7mb-F98GGdk 5ejto-7q7mb-F98GGdk

这个问题我们得加一些限制了:

  • 不支持拖拽文档中容器块下的标题
  • 不支持拖拽为自己的子标题

88250 added a commit that referenced this issue Apr 2, 2024
88250 added a commit that referenced this issue Apr 2, 2024
Vanessa219 added a commit that referenced this issue Apr 2, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
88250 added a commit that referenced this issue Apr 2, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
88250 added a commit that referenced this issue Apr 2, 2024
@88250 88250 added the Bug label Apr 2, 2024
@88250 88250 added this to the 3.0.7 milestone Apr 2, 2024
@88250
Copy link
Member

88250 commented Apr 2, 2024

撤销这个问题不太确定是不是上面其他问题引起的 #10828 (comment)

@88250
Copy link
Member

88250 commented Apr 2, 2024

层级6拖拽到层级3,会变成层级4

这个是设计如此,拖入到父级中后层级+1

@Vanessa219
Copy link
Member

在特殊情况下大纲能被带走(在文档树拖拽文档时则不会发生这种情况):

大纲3 大纲3

这个要怎么操作?

@88250
Copy link
Member

88250 commented Apr 2, 2024

要拖拽到同级大纲下才能拖拽成功

这个演示没太看懂,我这里可以将 H1 拖入到 H2 中的,或者拖为 H2 的平级。

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

在特殊情况下大纲能被带走(在文档树拖拽文档时则不会发生这种情况)

这个要怎么操作?

我是在拖拽的过程中用快捷键唤起 PixPin 截图

@88250
Copy link
Member

88250 commented Apr 2, 2024

要拖拽到同级大纲下才能拖拽成功

这个演示没太看懂,我这里可以将 H1 拖入到 H2 中的,或者拖为 H2 的平级。

现在懂了,不用再补充了,多谢。

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

要拖拽到同级大纲下才能拖拽成功

这个演示没太看懂,我这里可以将 H1 拖入到 H2 中的,或者拖为 H2 的平级。

现在懂了,不用再补充了,多谢。

我刚录完:

截图

@88250 88250 changed the title 大纲拖拽功能异常 Outline drag function is abnormal Apr 2, 2024
88250 added a commit that referenced this issue Apr 2, 2024
Vanessa219 added a commit that referenced this issue Apr 2, 2024
Vanessa219 added a commit that referenced this issue Apr 2, 2024
88250 added a commit that referenced this issue Apr 2, 2024
@88250 88250 closed this as completed Apr 2, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

@88250 什么时候发个dev版看看?

Vanessa219 added a commit that referenced this issue Apr 2, 2024
@88250
Copy link
Member

88250 commented Apr 2, 2024

我现在打一个,稍等。

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

还有这个要改一改:单纯点击时不应该拖拽

就拿文档树来说,左键点击文档的时候不会变成拖拽,鼠标挪动一点才会变成拖拽。

image

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

应该插入到标题与标题前一个块的中间,而不是把前面的块包进去:

标题

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

这里还有点问题:

拖拽

@88250
Copy link
Member

88250 commented Apr 2, 2024

应该插入到标题与标题前一个块的中间,而不是把前面的块包进去:

标题 标题

这个应该没有问题,因为之前的第一部分上方没有标题块,所以把标题块放到最上面以后就会包括它。

@88250
Copy link
Member

88250 commented Apr 2, 2024

这里还有点问题:

拖拽 拖拽

这个演示没看明白,能否帮忙用个简单的测试用例再演示一下,谢谢。

Vanessa219 added a commit that referenced this issue Apr 2, 2024
Vanessa219 added a commit that referenced this issue Apr 2, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

应该插入到标题与标题前一个块的中间,而不是把前面的块包进去:

这个应该没有问题,因为之前的第一部分上方没有标题块,所以把标题块放到最上面以后就会包括它。

我觉得不对,一般不会在开头放标题的,写文章开头一般先有一部分引言,然后才是第一个标题

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 2, 2024

这里还有点问题:

这个演示没看明白,能否帮忙用个简单的测试用例再演示一下,谢谢。

  1. 把 02 拖到 01 下变成子标题(没问题)
  2. 把 02 拖到与 01 同级的下方时,把刚才与 02 同级的 3、4、5 也一起带走了(有问题)
  3. 此时撤回,没有恢复到产生问题之前的状态(有问题)

@88250
Copy link
Member

88250 commented Apr 2, 2024

应该插入到标题与标题前一个块的中间,而不是把前面的块包进去:

这个应该没有问题,因为之前的第一部分上方没有标题块,所以把标题块放到最上面以后就会包括它。

我觉得不对,一般不会在开头放标题的,写文章开头一般先有一部分引言,然后才是第一个标题

也有道理,那就插入到第一个标题前面吧,同时把这个标题的级别和原来第一个标题的级别调整为一致的。

@88250
Copy link
Member

88250 commented Apr 2, 2024

这里还有点问题:

这个演示没看明白,能否帮忙用个简单的测试用例再演示一下,谢谢。

  1. 把 02 拖到 01 下变成子标题(没问题)
  2. 把 02 拖到与 01 同级的下方时,把刚才与 02 同级的 3、4、5 也一起带走了(有问题)
  3. 此时撤回,没有恢复到产生问题之前的状态(有问题)

懂了,多谢!

88250 added a commit that referenced this issue Apr 2, 2024
88250 added a commit that referenced this issue Apr 2, 2024
88250 added a commit that referenced this issue Apr 2, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 3, 2024

拖拽一次之后有段时间是用来排序的,此时进行下一次拖拽操作,在大纲刷新之后鼠标拖着的标题就看不见了:

阻止

@88250
Copy link
Member

88250 commented Apr 3, 2024

@Vanessa219 界面可能需要阻塞一下。

@Vanessa219
Copy link
Member

ok

Vanessa219 added a commit that referenced this issue Apr 3, 2024
@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 5, 2024

@Vanessa219 感觉应该还能再优化一下:

优化

优化

@Vanessa219
Copy link
Member

没看明白

@TCOTC
Copy link
Contributor Author

TCOTC commented Apr 9, 2024

  1. 标题正在排序的过程中,编辑文档会被吞字,排序的过程中应该阻止编辑文档
  2. 拖拽大纲之后,排序的过程中仍然可以拖拽大纲,但要等到排完序之后才显示拖拽的是哪一条。排序的过程中应该阻止拖拽

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants