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

Outline view is missing the keyboard command to just move the editor to the current a symbol #90732

Closed
chrisjj opened this issue Feb 15, 2020 · 8 comments · Fixed by #91799
Closed
Assignees
Labels
feature-request Request for new features or functionality good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes outline Source outline view issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@chrisjj
Copy link

chrisjj commented Feb 15, 2020

1 In Outline View, mouse-click on a symbol e.g,.
image
2 See the editor moves to that symbol
3 Press Down Arrow to move to another symbol.
4 Try to find a keypress that just moves the editor to that symbol.

Expected: success
Observed: failure, here. There's none I can find by experiment or reading docs. The nearest is Enter, but that also switches focus away.

Check the Command Palette. Neither this or any other commands to operate Outline View are present.

V1.42.1 on Windows.

@jrieken jrieken added feature-request Request for new features or functionality good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities outline Source outline view issues labels Feb 17, 2020
@jrieken jrieken added this to the Backlog milestone Feb 17, 2020
@lambainsaan
Copy link
Contributor

@jrieken I am working on this.

@jrieken
Copy link
Member

jrieken commented Feb 25, 2020

code pointer:

this._revealTreeSelection(newModel, first, focus, aside);

Something similar for focus events is needed

@bpasero
Copy link
Member

bpasero commented Jun 12, 2020

Verification: bind a key to the new list.selectAndPreserveFocus command to verify this ask.

@isidorn isidorn added the verified Verification succeeded label Jun 30, 2020
@isidorn
Copy link
Contributor

isidorn commented Jun 30, 2020

This does not really work in Outline, however it works just fine in other views. For example in the References view and the explorer. In outline the focus still gets passed even when executing list.selectAndPreserveFocus command.
However since the original ask was for Outline I am reopening this.

@isidorn isidorn reopened this Jun 30, 2020
@isidorn isidorn added verification-found Issue verification failed and removed verified Verification succeeded labels Jun 30, 2020
@jrieken jrieken removed this from the June 2020 milestone Jun 30, 2020
@jrieken jrieken removed verification-found Issue verification failed verification-needed Verification of issue is requested labels Jun 30, 2020
@bpasero bpasero removed their assignment Jun 30, 2020
@bpasero
Copy link
Member

bpasero commented Jun 30, 2020

@jrieken the only reason this does not work is this code:

this._revealTreeSelection(newModel, e.element, !e.editorOptions.preserveFocus || !!e.editorOptions.pinned, e.sideBySide);

If I remove || !!e.editorOptions.pinned it works fine. What is the rationale?

@jrieken
Copy link
Member

jrieken commented Jun 30, 2020

What is the rationale?

idk @joaomoreno added this via bb5b4e9

@joaomoreno
Copy link
Member

Yeah definitely an issue with bb5b4e9

With the fix, list.selectAndPreserveFocus should now work.

@joaomoreno joaomoreno added this to the June 2020 milestone Jul 1, 2020
@joaomoreno joaomoreno added the verification-needed Verification of issue is requested label Jul 1, 2020
@bpasero
Copy link
Member

bpasero commented Jul 1, 2020

Thanks!

@RMacfarlane RMacfarlane added verified Verification succeeded and removed verified Verification succeeded labels Jul 1, 2020
@bpasero bpasero added the verified Verification succeeded label Jul 2, 2020
@jrieken jrieken added the on-release-notes Issue/pull request mentioned in release notes label Jul 3, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Aug 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality good first issue Issues identified as good for first-time contributors help wanted Issues identified as good community contribution opportunities insiders-released Patch has been released in VS Code Insiders on-release-notes Issue/pull request mentioned in release notes outline Source outline view issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants
@joaomoreno @bpasero @jrieken @isidorn @RMacfarlane @chrisjj @lambainsaan and others