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

Bookmark explorer is slow with large-ish file #705

Open
Ben-Voris opened this issue Mar 26, 2024 · 1 comment
Open

Bookmark explorer is slow with large-ish file #705

Ben-Voris opened this issue Mar 26, 2024 · 1 comment

Comments

@Ben-Voris
Copy link

Environment/version

  • Extension version:
  • VSCode version: Version: 1.87.2 (system setup)
    Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
    Date: 2024-03-08T15:20:17.278Z
    Electron: 27.3.2
    ElectronBuildId: 26836302
    Chromium: 118.0.5993.159
    Node.js: 18.17.1
    V8: 11.8.172.18-electron.0
    OS: Windows_NT x64 10.0.19045
  • OS version: Windows 10 22H2

Steps to reproduce

  1. Open a large-ish file that has a reoccurring string that you can search for and then bookmark. I ran into this with a file that has 24 hits (the first on line 12 and the last on line 90200) in 90300 lines. The file is 5.5 MB.
  2. Find the string and bookmark the line.
  3. Add a bookmark. Make the BOOKMARKS EXPLORER visible.
  4. Repeat 2.

After adding a bookmark, the Bookmarks Explorer does not update for several seconds. Navigating to a bookmark from the Bookmarks Explorer takes 3 to 5 seconds.

Using Bookmarks: List is usually much faster.

Also, because a search for the string I'm bookmarking immediately comes up with the number of hits (the "x of y" next to the search string), I'm convinced the problem isn't with VS Code or the size of the file.

@alefragnani
Copy link
Owner

Hi @Ben-Voris ,

I have worked in the past with a few large files, but not that large, withing folders with thousands of files, but didn't notice any performace issue. I'll try to reproduce this scenario, in oder to replicate the issue, and find out the reason behind this performance issue.

About the comparation with Bookmarks: List commad, it will always be faster, compared with the Side Bar. Simply because the Side Bar displays the bookmarks from the entire folder, and the Bookmarks: List command does for the active file. A better comparation would be with the Bookmarks: List from All Files command instead.

Thanks in advance

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

No branches or pull requests

2 participants