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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Markers viewlet goes empty when being dragged #96413

Closed
jrieken opened this issue Apr 28, 2020 · 2 comments
Closed

Markers viewlet goes empty when being dragged #96413

jrieken opened this issue Apr 28, 2020 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@jrieken
Copy link
Member

jrieken commented Apr 28, 2020

while verifying #94530

Screen Recording 2020-04-28 at 15.41.37.mov.zip

  • see attached recording 馃敐
  • show markers
  • have file with errors
  • drag marker to sidebar
  • drag back to bottom panel
  • 馃悰 panel is empty, content is gone
@sbatten
Copy link
Member

sbatten commented Apr 28, 2020

okay, this is the third issue around moving a default view container to and from the sidebar causing issues. have a feeling there is a singular issue in the moving view containers logic.

@sbatten sbatten added the bug Issue identified by VS Code Team member as probable bug label Apr 28, 2020
@sbatten sbatten added this to the April 2020 milestone Apr 28, 2020
@sandy081
Copy link
Member

Looks like panel composite is not removed on deregister - https://github.com/microsoft/vscode/blob/master/src/vs/workbench/browser/parts/panel/panelPart.ts#L225

Hence ViewPaneContainer.create is not called when view container is moved back to panel.

@jrieken jrieken added the verified Verification succeeded label Apr 30, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jun 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants