[GH-ISSUE #2952] [Bug] The animation of scrolling to heading is too slowly when using outline #1293

Closed
opened 2026-03-23 20:47:31 +00:00 by mirror · 11 comments
Owner

Originally created by @LucasXu0 on GitHub (Jul 7, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2952

Bug Description

As the title said, the animation is too slowly now.

  • Just remove the animation is fine.

How to Reproduce

  1. insert some headings.
  2. insert an outline block.
  3. click the content inside the outline.

Expected Behavior

Scroll the position of heading directly without any animation.

Operating System

all platform.

AppFlowy Version(s)

0.2.5

Screenshots

No response

Additional Context

No response

Originally created by @LucasXu0 on GitHub (Jul 7, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2952 ### Bug Description As the title said, the animation is too slowly now. - Just remove the animation is fine. ### How to Reproduce 1. insert some headings. 2. insert an outline block. 3. click the content inside the outline. ### Expected Behavior Scroll the position of heading directly without any animation. ### Operating System all platform. ### AppFlowy Version(s) 0.2.5 ### Screenshots _No response_ ### Additional Context _No response_
Author
Owner

@emmggi commented on GitHub (Jul 7, 2023):

This slow scrolling also happens when your cursor leaves the screen by scrolling and then start typing. The animation is very slow to bring you back to the cursor. I thought these were purposely implemented.

<!-- gh-comment-id:1625991762 --> @emmggi commented on GitHub (Jul 7, 2023): This slow scrolling also happens when your cursor leaves the screen by scrolling and then start typing. The animation is very slow to bring you back to the cursor. I thought these were purposely implemented.
Author
Owner

@AmanNegi commented on GitHub (Jul 10, 2023):

@LucasXu0 I would like to fix this issue with the Outline.

<!-- gh-comment-id:1628021304 --> @AmanNegi commented on GitHub (Jul 10, 2023): @LucasXu0 I would like to fix this issue with the Outline.
Author
Owner

@LucasXu0 commented on GitHub (Jul 10, 2023):

@AmanNegi Do you have any ideas now? Feel free to share them here, and we can discuss them together.

<!-- gh-comment-id:1628676407 --> @LucasXu0 commented on GitHub (Jul 10, 2023): @AmanNegi Do you have any ideas now? Feel free to share them here, and we can discuss them together.
Author
Owner

@johansutrisno commented on GitHub (Jul 18, 2023):

Hi @LucasXu0 does it meet the requirements? I sped up the animation so the user doesn't feel too slow

https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/5f729a63-e2f3-4ec8-9573-f35bbace2b2a

<!-- gh-comment-id:1641053172 --> @johansutrisno commented on GitHub (Jul 18, 2023): Hi @LucasXu0 does it meet the requirements? I sped up the animation so the user doesn't feel too slow https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/5f729a63-e2f3-4ec8-9573-f35bbace2b2a
Author
Owner

@LucasXu0 commented on GitHub (Jul 19, 2023):

Hi @LucasXu0 does it meet the requirements? I sped up the animation so the user doesn't feel too slow

Screen.Recording.2023-07-19.at.05.08.21.mov

I suggest removing the animation in this case. If you have any ideas, feel free to DM me(Lucas@AppFlowy) on Discord.

<!-- gh-comment-id:1641293854 --> @LucasXu0 commented on GitHub (Jul 19, 2023): > Hi @LucasXu0 does it meet the requirements? I sped up the animation so the user doesn't feel too slow > > Screen.Recording.2023-07-19.at.05.08.21.mov I suggest removing the animation in this case. If you have any ideas, feel free to DM me(Lucas@AppFlowy) on Discord.
Author
Owner

@Xazin commented on GitHub (Jul 19, 2023):

I also believe we should remove the animation for the best UX.

<!-- gh-comment-id:1641798672 --> @Xazin commented on GitHub (Jul 19, 2023): I also believe we should remove the animation for the best UX.
Author
Owner

@AmanNegi commented on GitHub (Jul 19, 2023):

Yes in a long document this type of scrolling would not be a good experience. If possible we can add a curve to the animation like easeInOut. So that it feels more natural to the user.

<!-- gh-comment-id:1642117946 --> @AmanNegi commented on GitHub (Jul 19, 2023): Yes in a long document this type of scrolling would not be a good experience. If possible we can add a curve to the animation like easeInOut. So that it feels more natural to the user.
Author
Owner

@johansutrisno commented on GitHub (Jul 21, 2023):

I see, is it better than before?
cc @LucasXu0 @Xazin @AmanNegi

https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/278290f2-ba38-494f-b9cf-b5d2dd3eeefd

<!-- gh-comment-id:1645890550 --> @johansutrisno commented on GitHub (Jul 21, 2023): I see, is it better than before? cc @LucasXu0 @Xazin @AmanNegi https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/278290f2-ba38-494f-b9cf-b5d2dd3eeefd
Author
Owner

@Xazin commented on GitHub (Jul 21, 2023):

I see, is it better than before?

cc @LucasXu0 @Xazin @AmanNegi

https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/278290f2-ba38-494f-b9cf-b5d2dd3eeefd

Yeah, best for now. 🙏

<!-- gh-comment-id:1646113569 --> @Xazin commented on GitHub (Jul 21, 2023): > I see, is it better than before? > > cc @LucasXu0 @Xazin @AmanNegi > > > > https://github.com/AppFlowy-IO/AppFlowy/assets/22239637/278290f2-ba38-494f-b9cf-b5d2dd3eeefd > > > > Yeah, best for now. 🙏
Author
Owner

@johansutrisno commented on GitHub (Jul 22, 2023):

okay I will create the PR ASAP 🙏🏻

<!-- gh-comment-id:1646649615 --> @johansutrisno commented on GitHub (Jul 22, 2023): okay I will create the PR ASAP 🙏🏻
Author
Owner

@johansutrisno commented on GitHub (Jul 22, 2023):

I have created the PR for appflowy repo and appflowy-editor repo, please review the PR, thank you in advance

<!-- gh-comment-id:1646669103 --> @johansutrisno commented on GitHub (Jul 22, 2023): I have created the PR for appflowy repo and appflowy-editor repo, please review the PR, thank you in advance - https://github.com/AppFlowy-IO/AppFlowy/pull/3045 - https://github.com/AppFlowy-IO/appflowy-editor/pull/337
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#1293
No description provided.