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

🐞 Sidebar animation is not smooth #1041

Closed
jacob-carlborg opened this issue Feb 7, 2023 · 6 comments
Closed

🐞 Sidebar animation is not smooth #1041

jacob-carlborg opened this issue Feb 7, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@jacob-carlborg
Copy link

jacob-carlborg commented Feb 7, 2023

Description

The animation for hiding and showing the sidebars is not smooth. It applies to both the left and the right sidebar (inspector). It seems to be related to the size of the currently opened document. If no document is open, it's smooth. If a large document is opened the animation is not smooth. The larger document, the less smooth the animation is. It's enough with a document of 240 lines to notice a difference. If the document is over 1000 lines it's clearly visible. See attached video which shows the problem when a document with slight more than 1000 lines is open:

codeedit_sidebar.mov

To Reproduce

  1. Open a project
  2. Open a large document (1000 lines or more)
  3. Click the show/hide button for the inspector pane/right sidebar

Expected behavior

I expect the animation to be as smooth as in Xcode. See attached video for how it looks like in Xcode:

xcode_sidebar.mov

Version information

CodeEdit: 0.0.1-alpha (25)
macOS: 13.0.1 (22A400)

Additional context

No response

@jacob-carlborg jacob-carlborg added the bug Something isn't working label Feb 7, 2023
@matthijseikelenboom
Copy link
Contributor

@thecoolwinter I suspect this has to do with this CodeEditTextView issue. What do you think?

@thecoolwinter
Copy link
Collaborator

Most likely, it's a known issue with the ruler view being very inefficient for resizes as it redraws every line number every update.

@austincondiff
Copy link
Collaborator

Would this be solved when we implement our own ruler view then via https://github.com/CodeEditApp/CodeEditTextView/issues/127?

@austincondiff austincondiff changed the title Sidebar animation is not smooth 🐞 Sidebar animation is not smooth Feb 8, 2023
@thecoolwinter
Copy link
Collaborator

It should be, also related is CodeEdit/CodeEditTextView#131 .

@thecoolwinter
Copy link
Collaborator

Actually, it looks like Marcin did a bunch of work on STTextView recently, this might be fixed with CodeEditApp/CodeEditTextView#138.

@thecoolwinter
Copy link
Collaborator

Yep! Fixed in the previously mentioned PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏁 Complete
Development

No branches or pull requests

4 participants