Skip to content

feat: allow webviews in secondary side bar for extensions #247899

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

CamiKubaSilva
Copy link

Add support for contributing views to the secondarybar location

Summary

This PR enables extensions to contribute view containers and views to the Secondary Side Bar (secondarybar) using the existing viewsContainers and views contribution points.


What this change does

  • Adds "secondarybar" as a valid location for viewsContainers.
  • Updates the extension point schema and handler logic to register view containers and views under ViewContainerLocation.AuxiliaryBar.
  • Allows contributed views to be displayed in the Auxiliary Bar (aka Secondary Side Bar).

Before

Only "activitybar" and "panel" were accepted locations for contributed view containers:

"viewsContainers": {
  "activitybar": [...],
  "panel": [...]
}

Extensions could not target the Secondary Side Bar for contributed views.


After

Developers can now contribute view containers to the secondarybar:

"viewsContainers": {
  "activitybar": [...],
  "panel": [...],
  "secondarybar": [
    {
      "id": "my-view",
      "title": "My View",
      "icon": "media/icon.svg"
    }
  ]
}

Views inside this container will be rendered in the Auxiliary Bar.


Motivation

Allowing extensions to render in the Secondary Side Bar makes it possible to separate content logically—for example, Explorer in the primary sidebar and a custom Assistant in the secondarybar—without UI conflicts.

This increases flexibility and helps support more complex UI workflows in advanced extension scenarios.

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

Successfully merging this pull request may close these issues.

2 participants