Skip to content

Show pinned workbenches in the sidebar - #403

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-6657-pinning-a-workbench-has-zero-visible-effect-in-the-sidebar
Aug 24, 2026
Merged

TheGreatAxios merged 4 commits into
mainfrom
cl-6657-pinning-a-workbench-has-zero-visible-effect-in-the-sidebar

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Dispatch WORKBENCHES_MUTATED_EVENT after pin/unpin so the sidebar list refetches.
  • Show a pin glyph on pinned workbench rows and keep local pinned state in sync with props.

Closes CL-6657

Test plan

  • Existing tests green (context-menu pin event + workbench-list pin glyph)
  • Manually pin a workbench from the shell context menu and confirm sidebar reorders / shows pin
  • Unpin and confirm glyph clears

Dispatch WORKBENCHES_MUTATED_EVENT on pin so the list refetches, render a pin glyph on pinned rows, and sync local pinned state from props.

Closes CL-6657
After the coworker IA split, a single mixed list is the wrong pin
contract. The glyph test now covers both sections.
@TheGreatAxios
TheGreatAxios force-pushed the cl-6657-pinning-a-workbench-has-zero-visible-effect-in-the-sidebar branch from a1fc210 to 97aafec Compare August 24, 2026 19:41
@TheGreatAxios
TheGreatAxios merged commit 75137bb into main Aug 24, 2026
5 checks passed
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.

1 participant