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

Remove or add block causes editor to scroll #9891

Closed
3 tasks done
heheda123123 opened this issue Dec 15, 2023 · 13 comments
Closed
3 tasks done

Remove or add block causes editor to scroll #9891

heheda123123 opened this issue Dec 15, 2023 · 13 comments
Assignees
Labels
Milestone

Comments

@heheda123123
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Can the issue be reproduced with the default theme (daylight/midnight)?

  • I was able to reproduce the issue with the default theme

Could the issue be due to extensions?

  • I've ruled out the possibility that the extension is causing the problem.

Describe the problem

录了个视频如下
对于某些旧文档,只要回车,或者删除换行,都会触发翻页,普通输入文本不会翻页
我进行了如下测试
1 新建一个文档,复制这个文档的全部内容过去,不会有这种bug
2 找其他老文档,也不会触发这个bug
1702629432828.zip

Expected result

不翻页

Screenshot or screen recording presentation

No response

Version environment

- Version: 2.11.2
- Operating System: windows 11
- Browser (if used):

Log file

应该不用

More information

No response

@heheda123123 heheda123123 changed the title 编辑旧文档时回车就会翻页 编辑旧文档时换行相关的操作会触发翻页 Dec 15, 2023
@heheda123123 heheda123123 changed the title 编辑旧文档时换行相关的操作会触发翻页 编辑某些旧文档时换行相关的操作会触发翻页 Dec 15, 2023
@Vanessa219
Copy link
Member

可以把这个能重现的老文档发一下么?我导入看看是否能重现。谢谢

@Vanessa219
Copy link
Member

过 16s 后输入还会这样么?

@heheda123123
Copy link
Author

可以把这个能重现的老文档发一下么?我导入看看是否能重现。谢谢

我去,我刚删了。不过我又找到另一个文档有这个问题
msys2.sy.zip

@heheda123123
Copy link
Author

咦,不过msys2这个文档我导出之后又无法复现了,我也不知道是导出过,还是过了一段时间造成的
不过之前那个文档,等了很长的时间也还是存在这个问题,我再找找其他文档有没有这个情况的

@heheda123123
Copy link
Author

heheda123123 commented Dec 15, 2023

还真是,我又找到一个文档有这个bug。不过过了差不多十几秒之后就没这个情况了

@Vanessa219
Copy link
Member

哦,那我知道了。上个版本改进过,由于图片、图表、数学公式等加载会导致才打开的页面定位不准确,所以 16s 内高度变化的话都会重新定位。我把时间再缩短一点看看吧。主要想着才打开没有人会回车或删除之类的操作。

@heheda123123
Copy link
Author

heheda123123 commented Dec 15, 2023

不过之前那个文档我很确定,不止十几秒,是一直有这个问题,因为我写几行,就得往上滚动回到原位置,反复操作了很久。
搞得很难受。
是不是某种情况,16s会重新计时
我录的视频里面就有14s,在这之前一直有这个情况

@Geo123abc
Copy link

这个bug有点紧急啊,很影响编辑

@Vanessa219
Copy link
Member

动态滚动的时候是有一点问题,感谢反馈,马上进行处理。

Vanessa219 added a commit that referenced this issue Dec 15, 2023
@Vanessa219 Vanessa219 self-assigned this Dec 15, 2023
@Vanessa219 Vanessa219 added the Bug label Dec 15, 2023
@Vanessa219 Vanessa219 added this to the 2.11.3 milestone Dec 15, 2023
@Vanessa219 Vanessa219 changed the title 编辑某些旧文档时换行相关的操作会触发翻页 Remove or add block causes editor to scroll Dec 15, 2023
EightDoor added a commit to EightDoor/siyuan that referenced this issue Dec 15, 2023
* commit '969a8d24fcdc648b3f88d28d47280869f4974396':
  🎨 Improve web page clip code block content conversion siyuan-note#9896
  🐛 Create doc failed on non-Windows siyuan-note#9890
  🐛 fix siyuan-note#9850
  🎨 Add Relation and Rollup column to database table view siyuan-note#9888
  🐛 Do not remove .tmp file when listing doc tree https://ld246.com/article/1702621572239
  ♻️ Refactor database table view
  🐛 fix siyuan-note#9891
  🎨 delete row by keydown
  🎨 fix siyuan-note#9886
  🎨 siyuan-note#9886
  🎨 Enable or disable all CSS and JS code snippets siyuan-note#9860
  🎨 Database table view template columns support reference date columns siyuan-note#9887
  🎨 fix siyuan-note#9878
  ⬆️ Upgrade kernel deps
  🎨 Delete the corresponding data in the database attributes table after deleting the doc siyuan-note#9875
  ⬆️ Upgrade Lute
  🎨 Add internal kernel API `/api/block/getTailChildBlocks` siyuan-note#9884
  🎨 siyuan-note#9882
  🐛
@IAliceBobI
Copy link

@Vanessa219 v2.11.3

现在还有这个问题。但我也不知道规律。

回车就往下跳。

一开始以为是自己插件搞的,但禁用全部插件后依然能复现。

下面是复现的步骤,但不是稳定复现。我也不知道规律。

找一个长文档,要分段加载的。
在中上部定位,我是打个空白行。
关闭文档。
ctrl+shift+I , F5
ctrl+e 找到刚才的文档,在之前位置点击。
ctrl+end , 在末尾几个空行中随便敲个回车。有时会跳到好远上面。

有时ctrl+end后,再用滚轮定位到上面,找个地方敲回车也会跳,向下跳。

总之,就是随意多试试几次。有时随意的:F5,上下滚动,回车组合也触发。不分文档,长的不能一次加载完的就可以。

@Vanessa219
Copy link
Member

F5 上下滚动: a3ec74f
回车组合指的是 shift+Enter?
ctrl+shift+I 的功能是?

跳过 ctrl+shift+I 按照上面的步骤操作无法重现。

@IAliceBobI
Copy link

IAliceBobI commented Dec 21, 2023

@Vanessa219 不好意思,没说具体。 ctrl+shift+i 组合 f5就是要重新加载页面。把开发者工具调出而已,在js日志的地方 f5就刷新,重新加载了。

重新加载就是为了从一个新的状态开始尝试。

这个确实不好复现。目前我也找不到规律,先这样,等我发现了告诉您。

@Vanessa219
Copy link
Member

好的,谢谢。

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

4 participants