-
Notifications
You must be signed in to change notification settings - Fork 30k
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
How can we move the new side panel to the right? #141349
Comments
pinging @sbatten (bot made an assignment which surprised me) |
The new panel is limited to being placed opposite the side bar. I will leave this issue open to gather feedback. |
@webdev03 Could you upvote? People deserve options. |
See https://stackoverflow.com/a/70982514/836330 for a partial "workaround" in the meantime. |
As noted in #142075, these arbitrary changes are baffling in that they remove functionality we have been used to for years without solving any issue faced by users. It's unclear who decided on the changes and why, it looks like changing it for the sake of it. In more details
The whole design decision is a complete train wreck. I urge you to reconsider. Thanks. |
@gotofritz on this particular point I assume you know you can assign whatever shortcut suits you: |
For me this only happens after I set This capability arrived last year (see https://code.visualstudio.com/updates/v1_58#_terminals-in-the-editor-area). |
Yes of course. I wanted to flag that changing an existing shortcut to toggle a (potentially) empty panel, while the expected functionality is left to individual users to wire up manually in the preferences, is not ideal IMHO |
You are right, thanks for the tip... there are so many obscure terminal preferences settings that I may well have changed that manually in my desperate quest to bring back the old functionality |
Putting terminals in a dedicated editor group on one side could be improved by addressing this issue: #16927 - Allow a maximized editor group to use the full width rather than leaving 220px per group for each of the other editor groups. Also make |
@sbatten as I've stated elsewhere this is not a missing feature. It is a broken feature thus a breaking change in |
Upvoted, although this shouldn't be up for voting. This whole thing should be rolled back. |
This is an absolute joke, how is it possible to make such a regression in your softwares functionality! This feature has been working for years. |
Apparently no-one who uses that functionality does so with Insiders. Or if they do, I haven't seen an issue raised about it ahead of 1.64 shipping as Stable yesterday. Workaround for the moment is to revert to 1.63. Installation links are at https://code.visualstudio.com/updates/v1_63 and you may also want to turn off update checks. I am confident that the team (of which I am not a member) will be taking this feedback seriously. |
I'd like to add some general and specific thoughts here. General thoughtsIn general, having an additional panel might be useful for some users. Adding such a new panel, however, should not break anyone's workflow who does NOT need another panel. In an ideal case, users who don't need it should not even recognize that it has been introduced. In a semi-ideal case, they should at least be able to re-configure their previous workflow in the new environment. Both cases are not true for the introcution of the side panel, which makes this a breaking change and should be announced accordingly, in my opinion. Missed opportunitiesI assume that the developers didn't put enough effort into evaluating existing workflows and maybe didn't even notice the impacts of this change. It could have easily been designed non-breaking, e.g. as follows:
I don't see why the new side panel has been designed so different from the already existing panel. If I needed another panel, why force a specific layout? Who says I couldn't need two side panels? One at the left and one at the right? Or two bottom panels? However, now that the new side panel has been introduced, it might be too late to design it this way, because that would make it another breaking change. I still think that this would be the most flexible, least surprising way, though. The workaroundI don't think that the proposed "workaround" by putting the terminal into an editor group on the right side can replace previous workflows. The missing possibility to maximize the terminal has already been noted here. However, there are more limitations:
Proposed stepsTaking into account all the above, I see two options in order to make previous workflows possible again:
The bottom lineOf course the fix solution would be the easier one, but in my opinion it is far less sustainable as it bears the risk of future insonsistencies. From a developer's perspective, having two differently designed UI objects providing essentially the same functionality just leads to a higher maintenance effort due to more corner cases. |
I don't think that this is a valid criteria for introducing breaking changes. This is similar to the reasoning that was in The Hitchhiker's Guide to the Galaxy when the Vogons came to demolish Earth.
|
Note that #141569 is very closely related to this issue, in fact, that bug was introduced with the same change. If you vote for this please also vote for that. |
@gotofritz No, it was not changed "for the sake of it". Removing the ability to move the panel to the right was an unfortunate side-effect introduced by #132893. In fact the ability to move the side-panel to the same side of the explorer panel was never removed, because this side-panel did not exist. The previous panel in the bottom was capable to be moved to the side but this concept was ditched and merged with the new side-panel and a mistake was made. As @sbatten pointed out, the new side-panel was limited to the opposite side of the explorer which introduced this issue. |
What the fluff, man. They just told us that it gets fixed in the next patch. |
I just received v1.64.2 which has this fix. |
Thank you for the quick fix. 🙂 Unfortunately this fix only allows to move the bottom panel to the right, but I would also enjoy to move the new side panel to the same side like the explorer. #142819 |
I just wanted to share my expression and frustration of how bad the whole idea was. Have being using VSCdoe for 7+ years in my 12+ years developing experience. And this single change has been one of the most annoying in my whole IDE experience. It simply broke my keybinds and the most used workflow in the middle of last 3 days of release. Why didn't you go further and add Left Panel, Right Panel, Top Panel and Bottom Panel. So we can have panels over the panels around code editor and move panels into panels. This should be reverted completely and made an optional feature. P.S. Changes like this - are core usability changes. They should be added as optional for those that need them. Not enforcing it to the whole community. |
@h3d0 maybe not a good time to have updated VS Code, particularly if you haven't also been using Insiders alongside Stable. |
Thank you for the fix 1.62.2. I am not really sure what is the difference between Panel (which can be put on the side/right) and Side Panel (on the side). After the change the panels got messed up, also Ctrl+J my most-used keybind started producing new lines in Bash terminal (instead of toggling the Panel which was now Side Panel, because Panel could not be put at the side effectively). After 1.62.2 I was able to:
Everything is back to normal. |
@h3d0 I can see how this caused some confusion in the first place. Please take a look at #26777 to see why the side panel was added, so things start to make sense to you hopefully. Overall it was a good change and I hope you can see it the same way. The new side-panel is purely optional and you are able to continue to use VSCode like you did before, due to some reverting changes of 1.62.2 as you just figured out. The new side panel was never meant to be a replacement for your bottom panel that you've put to the right previously. It's meant for more flexibility, i.e. putting Outline on one side while having the Explorer on another and other useful scenarios. |
I totally agree with all your points, especially this one. I sat down one day to work on my project just to find that my workflow is totally broken. I've used dozens of editors throughout the years (Eclipse, Netbeans, Emacs, Idea, Vim, Notepad++ just to name a few) and this has never ever happened with any of them. I thought for a while that VS Code was the perfect editor. I no longer think that. The dream is dead. The worst part is not that when I start it I can expect this to happen but how it was handled. No one from the team acknowledged that this was a mistake, no one apologized and no one came here and said sorry guys, we messed up, not gonna happen again. I thought that Microsoft had very high standards, but apparently, they don't. |
@adam-arold Well, that's not true. @sbatten did that, and he provided a fix for the issue. And while I can understand any anger about this and Microsoft in general (I was annoyed by the change as well): let's not forget that the developers are still humans. Repeatedly posting how annoyed you are probably won't increase anyone's motivation to address your thoughts. |
That is simply not true: #141349 (comment) |
@adam-arold that's not fair. I think their response and actions were quite fast and everything is working as it was before now. And they actually did apologize to the users for this. To me, it looks like you have some generalized frustration towards Microsoft and their products. I actually thank every VSCode developer and contributor because they all made this amazing editor/IDE. And as stated above, we are humans and mistakes can happen and as long those mistakes get addressed and fixed it's fine; this is not a paid product, it's an open source project regardless if Microsoft may be having some of their developers to work on it. Thank you @sbatten for working on this and fixing it. |
Oh come on guys, it's true that @sbatten didn't cover himself in glory early on, but he learnt from his mistakes and the issue is now fixed (thanks!). Do we really need to keep going on about it? Maybe it's time to lock the thread.... |
You're right, this might be the case. I stand corrected. |
I complained on here as loud as I felt appropriate. After the fix came in I just wanted to come back and say thank you, and now I see people still piling on. Software lesson: After convincing the devs that it's a bug and they need to fix it RIGHT NOW, and then they fix it, you look like you don't understand software when you then steer the conversation toward the feature that caused the bug. That feature is a separate conversation, that you probably don't understand if you are still complaining in this thread. Life lesson: When a human makes a mistake and (after some dialogue) apologizes and fixes their mistake, it is insensitive, emotionally-tone-deaf, and probably inhumane to keep going on about what a bad idea it was. |
Version 1.62.2 fixed the issue and it became available faster than any expectation, after show the situations. Thanks yall involved. |
Or maybe the person in question has autism and you haven't considered that as an alternative to being insensitive, emotionally tone-deaf and inhumane. |
How exactly is it fixed? P.S. Please do not tell me off for being insensitive for daring to complain about it. |
@liborty have you read the comments please? :) This issue was closed in favor of #142742 and #142832. I think this issue should be locked. |
I don't believe it is a setting. I would like it to be so I can default open the terminal in the same place across all environments. And you would be forgiven for spending your efforts looking in the settings because so many other things about the UI can be changed in settings. Right-click toward the top of the terminal next to the tabs |
Thanks. I was stuck in catch 22. Those options were not showing because I did not activate them with "move views from side panel to panel". In the 'commands' rather than the 'settings' but also unrelated to the terminal. On the other hand, once I finally discovered this command, I now no longer need them. In addition, the side panel mysteriously can not be moved because someone somewhere made an arbitrary rule about it having to be opposite the side bar, which of course the user does not know about. What a mess! |
* debug: bump js-debug * Resources lost their authority part (fix microsoft#141328) * Run OSS tool * Enable resourceurls (microsoft#140709) * Enable resourceurls * Add selection to resourceurls from trees * Rename identifier * Respond to PR feedback * Do not loop opver openEditors * cleanup @vscode/windows-registry * Bump node-fetch from 2.6.2 to 2.6.7 (microsoft#141307) Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.2 to 2.6.7. - [Release notes](https://github.com/node-fetch/node-fetch/releases) - [Commits](node-fetch/node-fetch@v2.6.2...v2.6.7) --- updated-dependencies: - dependency-name: node-fetch dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump node-fetch from 2.6.1 to 2.6.7 in /build (microsoft#141306) Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.7. - [Release notes](https://github.com/node-fetch/node-fetch/releases) - [Commits](node-fetch/node-fetch@v2.6.1...v2.6.7) --- updated-dependencies: - dependency-name: node-fetch dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fixes microsoft#141392 * Add check that the editor's ESM JS files can be loaded in a browser * Create from template options for configure tasks Fixes microsoft#140867 * web - only simplify `vscode-remote` workspaces (microsoft#141411) * web - use correct untitled workspace object (fix microsoft#141411) * Install playwright using `npx playwright install` * Fix microsoft#141326 * Most recent -> last link Fixes microsoft#141419 * Add ellipsis to open detected link Fixes microsoft#141416 * $ref '' in 'http://json-schema.org/draft-07/schema' can not be resolved. Fixes microsoft#141445 * Use `@playwright/test` instead of `playwright` * Document how to unset auto reply, support null explicitly Fixes microsoft#141456 * remove unnecessary async * fix microsoft#141435 * add shell integration message to zsh part of microsoft#141448 * add shell integration message to pwsh, fix microsoft#141448 * Fix spelling error * Install playwright deps * Remove redundant 0 from bold sequence * fix microsoft#141420 * Run headless on CI * bold pwsh shell integration text * Fix cwdFolder on Windows Fixes microsoft#141443 * Move common imports to `.eslintrc.json` * Avoid `*` and spell out all allowed node modules to have squiggles when importing `vscode` by accident * Don't change target because `webpack` can't handle `es2020`. * Prevent link highlight in markdown code blocks and spans (microsoft#140816) * Prevent link highlight in markdown codeblocks (microsoft#139770) * Handle inline codespan variants for markdown link provider (microsoft#139770) * Refactor codespan detection in markdown link provider (microsoft#139770) * Add back ESM-uncomment * Remove export * fully qualify kind * Remove no longer necessary rule (microsoft#140989) * Get rid of `null!` * Fixes microsoft#140884: removal rules only target rules that appear before * handle bash login args (microsoft#141467) * support case insensitive shell executables, add tests * update with runtime key changes fixes microsoft#141389 * fixes microsoft#141387 * remove unused import * Update Codicons: Add layout icons (Refs microsoft#137633) microsoft/vscode-codicons@a2e80bf * fixes microsoft#141384 * fixes microsoft#141421 * Tweak CES survey assignment * Ambiguity in comments for createFileSystemWatcher (fix microsoft#141494) * 🆙 distro * Fix extension tree dnd supported types Fixes microsoft#141418 * Avoid `[createInstance]` warnings in unit tests * handle extension resource * Adopt `TestInstantiationService.get` * Improve comments * Improve explanations * Only log server greeting to log file, not again to the console * Avoid a lot of newlines when running from source * Fixes use of model after it got disposed. * Bump node-fetch from 2.6.1 to 2.6.7 in /extensions/github (microsoft#141538) Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.7. - [Release notes](https://github.com/node-fetch/node-fetch/releases) - [Commits](node-fetch/node-fetch@v2.6.1...v2.6.7) --- updated-dependencies: - dependency-name: node-fetch dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * support login args on linux too * formatting and clean up * Refactors unicode highlight options * Fixes microsoft#141152 by not showing unicode highlights in the ruler or minimap. * Fix cwd setting in workspace file Fixes microsoft#136099 Co-Authored-By: Megan Rogge <merogge@microsoft.com> * update doc * fix microsoft#141366. Safari does not respect input::placeholder rules, set line height explicitly. * better warning when an extension depends on an API proposal that doesn't exist, microsoft/vscode-python#18363 * remove inlay hints config again, fixes microsoft#140225 * debug: bump js-debug * Fix microsoft#141393 * testing: add autosave before each test run * testing: add autosave before each test run Save the current project file before each test run so that the code logic is up to date when the test is run * debug: Add debug.saveBeforeTest - Fix microsoft#139125 * testing: add autosave before each test run * testing: adjust the "saveBeforeTest" to a suitable position * fix microsoft#141367. * Fix microsoft#141461 * Bump node-fetch from 2.6.1 to 2.6.7 in /extensions/github-authentication (microsoft#141537) Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.7. - [Release notes](https://github.com/node-fetch/node-fetch/releases) - [Commits](node-fetch/node-fetch@v2.6.1...v2.6.7) --- updated-dependencies: - dependency-name: node-fetch dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump node-fetch in /extensions/microsoft-authentication (microsoft#141536) Bumps [node-fetch](https://github.com/node-fetch/node-fetch) from 2.6.1 to 2.6.7. - [Release notes](https://github.com/node-fetch/node-fetch/releases) - [Commits](node-fetch/node-fetch@v2.6.1...v2.6.7) --- updated-dependencies: - dependency-name: node-fetch dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix microsoft#141368. Overflow. * re microsoft#141368. disable replace all if the curr match is not model. * fix microsoft#141371. * 💄 * fix microsoft#141370. Update buttons on find match state change. * update find unit test. * Move `tokenizationSupport` inside `TokenizationStateStore` (microsoft#140476) * Rearrange methods * Extract `ContiguousGrowingArray` * Improve names and documentation for `TokenizationStateStore` * Add setting to control repository scan * fix microsoft#141515. * Fix microsoft#141469 * update terminalLinkManager's processCwd when cwd changes with shell integration on (microsoft#141595) * Emit event with valid range * make terminal output readonly (microsoft#141601) * Only show link commands if a terminal has been created Fixes microsoft#141593 * Fixes microsoft#139877: Allow `DomScrollableElement` to use float scroll values * update doc * Change notebook markdown fold hint to a CellPart * Cell fold hint fixes Hide when in edit mode, simplify top calculation, fix height calc when in edit mode * compositePart should use createActionViewItem helper fixes microsoft#141399 * feedback * fix microsoft#141294 * fix Terminal: Cannot refresh property when process is undefined microsoft#141288 * Bumping eslint versions (microsoft#141121) * hBumping eslint versions - Bump eslint and typescript/eslint versions to latest - Fixing `code-no-unused-expression` for `x?.(b);` type statements - Fixed a few new semicolon errors from upgrade * Bump eslint parser in build * Fix eslint errors in d.ts * update layout state prefix fixes microsoft#141372 microsoft#141376 microsoft#141374 * Fix import path * Close microsoft#141465 * fixes microsoft#141398 * fixes microsoft#141404 * unused import * Add link to sort description * Close microsoft#141555 * refs microsoft#141426 * fixes microsoft#141432 * fixes microsoft#141552 * Instrument watermark opening * Adding video tutorials to get started * add customize layout icons refs microsoft#137633 * Remove stale comment microsoft#140878 is fixed * Startup dialog goes through the request service (microsoft#141592) * show error when inlay hints command fails, annotate error with source (e.g extension name), fixes microsoft#141588 * Fixes microsoft#138746. * - Check if URL is pointing to the right service - Set selective request and response headers - Update the scheme on the client * cache resourceUrlTemplate Uri * Fixes microsoft#69391: Reserve extra bottom padding when the hover has a horizontal scrollbar * fix microsoft#141643 * Revert "always set aria-selected, even for listbox" This reverts commit 1356392. * Fixes microsoft#141638: `-command` rules should only remove default rules * fix microsoft#141299 * Remove terminalLocation from api proposal list Fixes microsoft#141645 * Match z-index on parameter hints to the hover Fixes microsoft#140761 * Reveal panel/editor when running a recent command/dir Fixes microsoft#141619 * Ensure quick pick is closed on view output Fixes microsoft#141615 * Inconsistent single quotes use in --help (same for code --help) * Reduce duplication * Fixes microsoft#25221: Render line highlights for all cursors * server: port range is end inclusive, improve validation * fix typo * Use `display:none` when overview ruler is turned off * Better align simple options with `<input>`s * Fixes microsoft#140367: Take into account horizontal padding when layouting the editor * Update scripts to point to `tsc` one folder up * Fix microsoft#141331 * Update description (microsoft/vscode-remote-release#6184) * fix typos * fix microsoft#141633 * await open editor * Allow richer commands in get started entries * Update src/vs/workbench/contrib/terminal/browser/terminalInstance.ts Co-authored-by: Megan Rogge <megan.rogge@microsoft.com> * create process in an empty workspace when cwd is userHome (microsoft#141660) * Fixes microsoft#140490 by always using `cursor:text` on macOS instead of our custom cursor. Since macOS Catalina the native cursor has a white border which makes it work well on dark themes, making our custom cursor not necessary. Also, macOS High Sierra is no longer officially supported by Apple (ref microsoft#96946) * Update shell integration config description Fixes microsoft#141308 * --help should read better * tty for ./scipts/code-server * Support null in auto replies setting Fixes microsoft#141666 * Improve auto replies config more Part of microsoft#141666 * debug: bump js-debug * Update typescript-eslint/experimental-utils Fixes microsoft#141653 * Update Codicons: Add modes and menu bar microsoft/vscode-codicons@ee85be1 * tweak setting language * refs microsoft#137633 * testing: fix inline decoration overlap Fixes microsoft#138915 * Use fully qualified names in @link * Fix `undefined`s in text search results (microsoft#141522) * Revert "create process in an empty workspace when cwd is userHome (microsoft#141660)" This reverts commit a58b538. * copy & continue. Fixes microsoft/vscode-internalbacklog#2613 * fixes microsoft#141471 * Fix microsoft#141477 * readonly * type -> interface * Avoid lookbehind in regexp Fixes microsoft#141689 * Clean up unused method, re microsoft#139541 * 💄 re microsoft#141582 * fix microsoft#141686 * Disable windows adjustment for active indicator (Refs microsoft#140647) * Temporary window shift * Revert * Dont require steps to be defined Fixes microsoft#141603 * Fix microsoft#141434 * Fix microsoft#141439 * microsoft#141054 enable the tests and keep debugging * Revert "fix microsoft#141686" This reverts commit 4af3e14 Didn't seem to fix the issue * Bump distro * Revert "support file llink" This reverts commit 09108ba. * Trim characters off match instead to get key, fixes microsoft#141614 (microsoft#141818) * Fix microsoft#141910 * Revert "fix microsoft#140907" This reverts commit a81c3b0. * Ensure parameter hints z-index is lower than suggest Terminal screen element's z-index is 31 Fixes microsoft#141947 * Fix comments can't be updated (microsoft#142013) Fixes of microsoft#142012 * debug: fix action bar input being hidden (microsoft#142230) For microsoft#142010 * Revert "debug: fix action bar input being hidden (microsoft#142230)" (microsoft#142240) This reverts commit 4a4c17c. * Bump distro and version (microsoft#142419) * Disable auto replies by default Fixes microsoft#142341 * Latest update breaks due to proxy requirement. Fixes microsoft#142201 * debug: bump js-debug (microsoft#142531) * debug: fix action bar input being hidden (microsoft#142530) For microsoft#142010 * GVFS: can't open file (fix microsoft#142481) (microsoft#142499) * safeguard init exthost message (microsoft#142486) * Execution object should be able to be returned by getCellExecution at the time the change event is fired microsoft#142466 * Make sure we pass along windows style paths to pluginProbeLocations (microsoft#142420) * Fix bare math blocks with nested beging/end (microsoft#142136) Fixes microsoft#141905 This picks up microsoft/vscode-markdown-it-katex@b1ed14d * Fix quoting for tasks (microsoft#142400) Fixes microsoft#142196 * Bump version to 1.64.2 * refs microsoft#142176 (microsoft#142423) * Candidate/142146 (microsoft#142424) * refs microsoft#142146 * refs microsoft#142146 * Fix microsoft#142421 (microsoft#142433) POINTER_DOWN was stealing too many events. The old event handlers handled events such as MOUSE_UP, and it turns out we can just use POINTER_UP instead. * fixes microsoft#141349 (microsoft#142644) * fixes microsoft#141349 * additional handling of the panel position with panel alignment * chore: update package.json and yarn.lock From what I can tell, it looks like there aren't many changes here. cookie was added as a devDependency. We had that as a regular dependency already. I moved it to match theirs. They were also using 0.4.0 while we had 0.4.1. Hope that doesn't matter. The other big update was eslint to 8.7.0 from 6.8.0. That could cause breaking changes potentially. Something to look out for. * feat: update product.json I don't see anything changed here for us to worry about. This is mostly making sure we keep the right names on win32. * fix: dont use remote node in code-server.sh I am unsure if this was the right move but it appears that we commented this out previously and did not load remote node. I think this is because it leads to Node version discrepencies which we don't want. * feat: add getPreferredWebSchema network.ts It appears they added a new method to the RemoteAuthoritiesImpl called `getPreferredWebSchema` which returns either 'http' or 'https'. * chore: resolve conflicts in src/vs/code/browser/workbench/workbench-dev.html Not much to report here. Seems like nothing new. * chore: update src/vs/code/browser/workbench/workbench.html Based on what I can tell, they changed workbench.web.api to workbench.web.main for most of these static assets. I hope that is accurate. * fix: conflict in src/vs/platform/remote/browser/browserSocketFactory.ts I am not 100% if the new code is similar to ours. We had to patch this to work against the current path. Their new implementation may do the same thing but I couldn't tell from looking at it. We may want to revisit this later when we switch to patches. Hopefully we can remove this. * fix: src/vs/platform/remote/common/remoteAgentConnection.ts It appears they've added a new parameter to `createSocket` which takes a string for a debugLabel. Not sure how it's used or made available but it's interesting because it takes the `connectionTypeToString` which we're adding to the query. Makes me think we might be able to remove this patch if our backend can get access to the debugLabel somehow. * fix: src/vs/server/node/serverEnvironmentService.ts Seems like a bunch of new serverOptions were added, some deprecated. I am not sure how this will affect code-server. They added 'host' so I wonder if we could deprecate 'bind-address' on the code-server side and instead use host:port with VS Code. Something to consider. * fix: src/vs/workbench/browser/web.main.ts Nothing major here. Had to add a random property called `ctor` to the CodeServerAdditions class. Also used connectionTokenCookie instead of hard-coded string which is a nice addition. * fix: src/vs/workbench/browser/web.api.ts Clean up IWorkbenchConstructionOptions. That's mostly it here. * fix: src/vs/workbench/contrib/extensions/browser/extensionsViewlet.ts Added flexible:Height: true to the SyncDescriptor since we were missing it. Removed a block that showed the prevented the web view extensions from automatically hiding. I did this because upstream removed two properties it was using: this.hasInstalledWebExtensionsContextKey and this.updateInstalledWebExtensionsContext. I hope that was right. * fix: src/vs/server/node/server.main.ts I removed `args['server-data-dir']` from `REMOTE_DATA_FOLDER` since we don't have access to `args` here. The other change was the `{ mode: 0o700 }` added to the `fs.mkdirSync` call. Not sure if that's related to chmod 0700 or something else. * wip * fix: conflicts in src/vs/server/node/webClientServer.ts We resolved a few things here with the conflicts. * fix: src/vs/code/browser/workbench/workbench.ts We were able to remove a couple of our patches, which is awesome! Still have some things to test though. * fix: src/vs/server/node/remoteExtensionHostAgentServer.ts Moved our custom serverThemeService into the src/vs/server/node folder and added the missing _serviceBrand to the interface and the class to fix a ctor error. * fix: remove clientTheme from webClientServer * fix: also remove from html files * fix: add missing __metadata change * chore: clean up code comments * chore: clean up comments * chore: fix typo * wip: add notes for debugging folder query param * refactor: remove workspace & folder from code-server args * fixup!: clean up TODOS * revert(code-server.sh): match upstream Co-authored-by: Connor Peet <connor@peet.io> Co-authored-by: Benjamin Pasero <benjamin.pasero@microsoft.com> Co-authored-by: Henning Dieterichs <hdieterichs@microsoft.com> Co-authored-by: Alex Ross <alros@microsoft.com> Co-authored-by: Joao Moreno <joao.moreno@microsoft.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Alex Dima <alexdima@microsoft.com> Co-authored-by: Logan Ramos <lramos15@gmail.com> Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com> Co-authored-by: Martin Aeschlimann <martinae@microsoft.com> Co-authored-by: meganrogge <merogge@microsoft.com> Co-authored-by: Waqas Ali <waqas.abbasi@outlook.com> Co-authored-by: Matt Bierner <matb@microsoft.com> Co-authored-by: Tyler Leonhardt <me@tylerleonhardt.com> Co-authored-by: Megan Rogge <megan.rogge@microsoft.com> Co-authored-by: SteVen Batten <sbatten@microsoft.com> Co-authored-by: Miguel Solorio <miguel.solorio@microsoft.com> Co-authored-by: Harald Kirschner <hkirschner@microsoft.com> Co-authored-by: Sandeep Somavarapu <sasomava@microsoft.com> Co-authored-by: rebornix <penn.lv@gmail.com> Co-authored-by: Johannes Rieken <johannes.rieken@gmail.com> Co-authored-by: Hans <416870376@qq.com> Co-authored-by: Ladislau Szomoru <3372902+lszomoru@users.noreply.github.com> Co-authored-by: Jackson Kearl <jakearl@microsoft.com> Co-authored-by: Rob Lourens <roblourens@gmail.com> Co-authored-by: isidor <inikolic@microsoft.com> Co-authored-by: Christof Marti <chrmarti@microsoft.com> Co-authored-by: gregvanl <gregvanl@microsoft.com> Co-authored-by: Benjamin Tidor <benjamin@tidor.net> Co-authored-by: Raymond Zhao <raymondzhao@microsoft.com> Co-authored-by: Henning Dieterichs <notify.henning.dieterichs@live.de> Co-authored-by: SteVen Batten <6561887+sbatten@users.noreply.github.com>
We use to be able to execute
move panel right
for moving the terminal panels to the right, however in the latest version of vscode (not sure where it started) that cmd has been deprecated, now we can only move the panel from the left side or docked on the bottom.How can we move the panel to the right? im almost certain im doing something wrong
main content | terminal panels stacked vertically | explorer panels stacked vertically
the above UX is essential,
vscode error
Moving the panel with this command has been deprecated in favor of "Move Views From Panel To Side Panel" and "Move Views From Side Panel To Panel" for similar functionality.
The text was updated successfully, but these errors were encountered: