Skip to content

feat(sidebar): add open and close methods for sidebar app management - #2969

Open
codewithchai0605 wants to merge 2 commits into
Acode-Foundation:mainfrom
codewithchai0605:feat/sidebar-app-plugin-controls
Open

codewithchai0605 wants to merge 2 commits into
Acode-Foundation:mainfrom
codewithchai0605:feat/sidebar-app-plugin-controls

Conversation

@codewithchai0605

Copy link
Copy Markdown

Add public api to open and close sidebar from the plugin.

@github-actions github-actions Bot added the enhancement New feature or request label Oct 13, 2026
@greptile-apps

greptile-apps Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; the wide-screen close issue is fixed.

Summary

Adds plugin-facing open() and close() methods for sidebar apps.

  • Plugins can open the sidebar with a chosen app or close it.

Reviews (2) · Last reviewed commit: "feat(sidebar): update hide method to acc..." · Reviewed by Greptile

Comment thread src/sidebarApps/index.js Outdated
@codewithchai0605

Copy link
Copy Markdown
Author

@greptile

@bajrangCoder

bajrangCoder commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Why do we need separate open and close APIs?

Isn't toggle-sidebar command is enough? just call it using : acode.exec("toggle-sidebar");

@codewithchai0605

Copy link
Copy Markdown
Author

@bajrangCoder toggle-sidebar is good, but it only opens the sidebar and last opened sidebar apps. This change will allow the plugin to open a particular sidebarApp, not the last opened.
Ofcourse plugin developer can open the particular sidebarApp by doing localStorage.setItem("sidebarAppsLastSection", sidebarAppId) and then they can call toggle-sidebar. But I think this is a cleaner way to do so, that why I added this features. If not so useful then, I can close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

3 participants