Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix Navigator Key Navigation Explosion#1803

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@thecoolwinter
Copy link
Collaborator

Description

Fixes an issue discussed a few differenttimes with the navigator.

To reproduce:

  • Enable "reveal in project navigator" setting.
  • Select a file
  • Using the arrow keys, move up or down to a folder in the navigator
  • Eventually, the app will freeze.

This fixes that bug, and another one I found on the way.

In this PR:

  • Add a few more cases whereshouldSendSelectionUpdate in the navigator outline view needed to be set. In these cases, extra selection events were being sent back to the navigator when a tab was opened. This was one of the reasons for the freeze as a few files were being opened over and over again.
  • Removes a call toupdateSelection when receiving a notice that the directory monitor has updated. This was causing an issue where, when revealing a previously unloaded directory, the selection would jump back to the opened tab rather than staying on the folder being opened.
  • Cleans up some of the editor history code, adding a few methods and moving as much history code as possible to an extension.

Related Issues

Checklist

  • I read and understood thecontributing guide as well as thecode of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

Screen.Recording.2024-07-09.at.10.26.18.PM.mov

lin72h reacted with thumbs up emoji
@thecoolwinterthecoolwinter merged commit451d4c2 intoCodeEditApp:mainJul 14, 2024
@thecoolwinterthecoolwinter deleted the fix/history-explosion branchJuly 14, 2024 20:55
@austincondiffaustincondiff added the bugSomething isn't working labelJul 19, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@0xWDG0xWDG0xWDG left review comments

@FastestMolassesFastestMolassesFastestMolasses approved these changes

@tom-ludwigtom-ludwigtom-ludwig approved these changes

@austincondiffaustincondiffAwaiting requested review from austincondiff

@matthijseikelenboommatthijseikelenboomAwaiting requested review from matthijseikelenboom

@MarcoCarnevaliMarcoCarnevaliAwaiting requested review from MarcoCarnevali

@bombardier200bombardier200Awaiting requested review from bombardier200

@Wouter01Wouter01Awaiting requested review from Wouter01

Assignees

No one assigned

Labels

bugSomething isn't workingnavigator

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

🐞 Reveal in Project Navigator not working

5 participants

@thecoolwinter@0xWDG@FastestMolasses@tom-ludwig@austincondiff

[8]ページ先頭

©2009-2025 Movatter.jp