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.

Sign up for GitHub

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

Jump to bottom

Outline is broken in the toolbar actions #158855

Closed
sandy081 opened this issue Aug 23, 2022 · 6 comments · Fixed by #159721
Closed

Outline is broken in the toolbar actions #158855

sandy081 opened this issue Aug 23, 2022 · 6 comments · Fixed by #159721
Assignees
@mjbvz @justschen
Labels
bug Issue identified by VS Code Team member as probable bug editor-code-actions Editor inplace actions (Ctrl + .) insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone
September 2022

Comments

@sandy081
Copy link
Member

Testing #158636

image
@mjbvz
Copy link
Collaborator

mjbvz commented Aug 23, 2022

Sorry but what is the issue?

@sandy081
Copy link
Member Author

Please see outline of Learn more about JS/TS refactoring action in the bottom toolbar. It looks broken.

@mjbvz mjbvz added this to the August 2022 milestone Aug 23, 2022
@mjbvz mjbvz added the bug Issue identified by VS Code Team member as probable bug label Aug 23, 2022
@mjbvz
Copy link
Collaborator

mjbvz commented Aug 23, 2022

Thanks. Also moving to September because I want to fix this by moving the action bar outside of the main list

@mjbvz mjbvz modified the milestones: August 2022, September 2022 Aug 23, 2022
@justschen
Copy link
Contributor

@mjbvz @sandy081 I wasn't able to repro in insiders - how are you interacting with it (by clicking or with keyboard)?

@sandy081
Copy link
Member Author

Using keyboard, by TAB

@mjbvz mjbvz added the editor-code-actions Editor inplace actions (Ctrl + .) label Aug 30, 2022
mjbvz added a commit to mjbvz/vscode that referenced this issue Sep 1, 2022
@mjbvz
Refactor code action widget
a04905c 
Refactors the code action widget somewhat heavily. The goal was to move the documentation items out of the main list widget

While doing this, I also tried to clean up the code to be safer (less state) and more readable

Fixes microsoft#158855
Fixes microsoft#158854
@mjbvz mjbvz mentioned this issue Sep 1, 2022
Merged
mjbvz added a commit that referenced this issue Sep 1, 2022
@mjbvz
Refactor code action widget (#159721)
44c59c7 
Refactors the code action widget somewhat heavily. The goal was to move the documentation items out of the main list widget

While doing this, I also tried to clean up the code to be safer (less state) and more readable

Fixes #158855
Fixes #158854
@vscodebotnpa vscodebotnpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Sep 1, 2022
@meganrogge meganrogge added the verification-steps-needed Steps to verify are needed for verification label Sep 29, 2022
@rzhao271 rzhao271 added verified Verification succeeded and removed verification-steps-needed Steps to verify are needed for verification labels Oct 3, 2022
@rzhao271
Copy link
Contributor

rzhao271 commented Oct 3, 2022

I accidentally found a way to verify the issue.
If one presses Ctrl+. on a private class field, the code action widget comes up with the "Learn more" link.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mjbvz mjbvz

@justschen justschen

Labels
bug Issue identified by VS Code Team member as probable bug editor-code-actions Editor inplace actions (Ctrl + .) insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Milestone
September 2022
Development

Successfully merging a pull request may close this issue.

Refactor code action widget mjbvz/vscode
6 participants
@rzhao271 @sandy081 @mjbvz @meganrogge @justschen @vscodebotnpa