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

remove panel location options in favor of new side panel behavior #140742

Merged
merged 4 commits into from
Jan 14, 2022

Conversation

sbatten
Copy link
Member

@sbatten sbatten commented Jan 14, 2022

This PR removes the old concept of moving the panel part in favor of a simpler model of moving view/containers into the new side panel. It "deprecates old commands" by redirecting them to the new commands to ease the transition though we know usage is already quite low of this deprecated feature.

@sbatten sbatten self-assigned this Jan 14, 2022
@sbatten sbatten added this to the January 2022 milestone Jan 14, 2022
@sbatten sbatten merged commit ea6f9fb into main Jan 14, 2022
@sbatten sbatten deleted the removePanelLocation branch January 14, 2022 21:53
@samestep
Copy link
Contributor

In case anyone else stumbles upon this like I did today, here are the steps to move the terminal to the side using the new system:

  • open the terminal
  • run the command View: Focus into Side Panel
  • drag the "Terminal" tab into the side panel

@sbatten
Copy link
Member Author

sbatten commented Jan 18, 2022

@samestep thanks

alternatively, if you want all the traditional panels (terminal, output, debug, problems) on the side panel, you can right click the panel and choose Move Views to Side Panel

@samestep
Copy link
Contributor

@sbatten Ahh that's what I was looking for; thanks!

@lukecotter
Copy link

Is there a way to have that side panel on the right?

@sbatten
Copy link
Member Author

sbatten commented Jan 19, 2022

@lukecotter the behavior of the side panel is that it remains opposite of the side bar. Toggling the side bar position will move the side panel as well. F1 > Toggle Side Bar Position. It is not possible to stack the side panel and side bar horizontally anymore.

@lukecotter
Copy link

lukecotter commented Jan 19, 2022

Thanks for responding.

That's a shame. I used to like having the panel and side bar next to each other with the code pushed all the way left.

@sbatten
Copy link
Member Author

sbatten commented Jan 19, 2022

feedback noted. if your primary use case of the panel is terminal, perhaps terminal editors are a valid solution for you

@dpooleffdc
Copy link

A small issue with"Move Views to Side Panel" is the ability toggle the terminal.

When the terminal is in its default location underneath the editor you can press ctrl + ` to toggle it, if you move this to the side panel that keyboard shortcut no longer works. I have noticed there is a command to "Toggle Side Panel" which I can rebind and will achieve the same result just a shame the Toggle terminal command does not work the same in the different locations.

@happyteque
Copy link

happyteque commented Jan 20, 2022

That's a shame. I used to like having the panel and side bar next to each other with the code pushed all the way left.

So @sbatten let me get this right - you have "improved" the code by removing functionality that a lot of users (including me) were happy with? For ... reasons? Thanks 🙄 Who exactly came to the unilateral decisions that the terminal can only be on the side opposite the file browser? wtf

Sometimes I wish MS could just leave the thing alone, I am happy with the way it works, why constantly "enhance" it and confuse your users... we got work to do. I guess I'll now go and spend a couple of hours of my time reading up on what the shiny new "Side Panel" behaviour and "Terminal Editors" are... 😡

EDIT: I have read up on panels and terminal editors... it's an over complicated mess. Why have two different abstractions that do the same thing. Why force the users to get involved in understanding those concepts and configuring them. Basically you are trying to recreate Eclipse. Jesus. I miss the days when VSCode Just Worked

@happyteque
Copy link

happyteque commented Jan 20, 2022

Also, with Terminal Editors, the workbench.action.terminal.focus which was burned into my muscle memory cannot be used anymore. Is there an equivalent for Terminal Editors?

No no, Terminal Editors are NOT the same thing. They open as just another tab in your document history, unless you use the command palette to open it to the side. When you try to toggle between different documents it gets in the way. It's COMPLETELY DIFFERENT.

Please bring back the ability to have the actual Terminal Panel to the right of the file browser. Thanks.

@sbatten
Copy link
Member Author

sbatten commented Jan 20, 2022

@dpooleffdc I have just made a change to fix this behaviour. It will be available in tomorrow's insiders.

@cognominal
Copy link

What is the purpose of the side panel ? I can't see any documentation about it. I understand one drags views into it. But I don't know what are views in this context.

@dpooleffdc
Copy link

@dpooleffdc I have just made a change to fix this behaviour. It will be available in tomorrow's insiders.

Thank you @sbatten that is very much appreciated!

@sbatten
Copy link
Member Author

sbatten commented Jan 21, 2022

@cognominal the side panel is a new location to drop views analogous to the side bar and panel that already exist today. It lacks documentation since it is yet to be released into stable. however that should change soon. views are the things inside of the side bar and panel, e.g. timeline view, outline view, terminal, etc.

@cognominal
Copy link

cognominal commented Jan 22, 2022

@sbatten All this easy shuffling around of views is very nice, once one gets the hang of it.
Is this going eventually in the direction of supporting different layouts for different usages or different modes within the same workspace, including possibly having closed editor groups, invisible but still there ?
Speaking of mode, I am thinking of screen casting (as opposed to live sharing) which means choosing setting screen in low resolution mode and adopting in conséquence different layouts.

@sbatten
Copy link
Member Author

sbatten commented Jan 22, 2022

@cognominal thanks for the feedback! I believe this issue would be a good place to share your thoughts and express your desire for the feature #138263

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

Successfully merging this pull request may close these issues.

6 participants