Repository navigation
fix(web): pinned sidebar rows keep the pin still on hover - #150
Merged
Merged
Conversation
The pin marker sat left of the status slot, so it slid sideways when the hover actions widened that slot. It is now the trailing item, outside the swap. Unpinned rows also get a pin action in the hover group. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
adampeterhiggins
added a commit
that referenced
this pull request
Oct 21, 2026
* test(server): restore fork tests dropped by the upstream sync The upstream test cull in #415 removed fork-only coverage for subfolder session import, imported-session resume, unguarded Claude overage silence and MCP tool-name uniqueness across the fork's toolkits. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(web): restore the command palette's fork actions lost in the upstream sync Upstream sync #415 rebuilt the command palette and dropped the fork's actions and views, leaving their hooks, reducer cases, and shortcuts orphaned. Restores, in the new structure: - New thread from PR, branch, or issue; Import conversation… - Export transcript, Show agents, Run scheduled task now (single task or a submenu, plus the run-scheduled-task open intent) - Attach thread / Linear / Notion / Granola / Slack / Discord / GitHub issue / pull request / repository, and link/unlink for GitHub, Linear, Slack, Discord, Notion and Granola - Open/close split view and focus the other pane; show/hide sidebar tabs - The Needs attention submenu and its attentionInbox.open shortcut - Deleting the `>` that came from the file picker returns to file search - Thread search shows tab-group-name matches again - Restart agent session goes through useRestartAgentSession Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(web): restore composer fork behaviour lost in the upstream sync Upstream sync #415 split the composer and silently dropped several fork behaviours. Port them back into the new structure: - @ menu Chats tab (Files/Chats/Agents/Models): Files lists paths only, Chats lists sibling chat tabs then other threads, opens on Chats when only chats match, has its own empty states, and picking a chat tab captures its summary as a thread-tab chip at the caret. - Pasted, stash-restored and draft-restored chips keep checkout, repository, thread-tab, issue, Slack/Discord/Notion/Granola, subagent, link and model context; stashing clears the thread's snapshot records. - Chips inserted mid-message land at the insertion caret (pending snapshot). - Citation comments get Save as draft and the Add to queue label. - composer.saveDraft keybinding works again. - A quick-message submit waits for attachment uploads. - File rows whose bytes are being restored show "Restoring…". Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(web): only the focused split pane's composer answers stash and save-draft keys Restores the fork's split-pane guard dropped in the #415 sync. Both panes mount a composer with a window-level key handler, so without it one shortcut stashed or saved a draft in both panes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(web): restore fork timeline and file-link behaviour lost in the upstream sync The #415 sync split MessagesTimeline and ChatMarkdown into smaller files and dropped fork behaviour along the way. Port it into the new structure: - Fork into new tab on user messages (onForkFromMessage through the row context) - Tool-call and thought hover preview cards on collapsed work rows, with the expanded row and card sharing ToolCallDetails/toolCallDetailOutput - Copyable path breadcrumbs on collapsed and live tool rows - View diagnostics on failed turns - Workspace changed divider for moved threads - @handle and Attach to parent chat on the Subagent of divider - Linear/Granola monochrome brand marks for tool activity icons - File links follow the Open in default and extension rules, offer an Open with submenu and Open in T3 file viewer, and keep the registry mention hover - Restored scroll offsets stay inside rows that rewrapped while hidden Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(web): restore PR panel, PR page, sidebar pin and work-log behaviour lost in the upstream sync The upstream sync (#415) silently dropped several fork behaviours while upstream split the PR panel and page into smaller components. Port them back into the new structure: - PR panel: cite quotes from descriptions and comments (#329) and save a fix or selection as a thread draft (#353) again; the Guide and Code tabs only offer drafts for server threads. - PR header number, repository link and "Open on host" menu item follow the "Open links in" setting, and the number's context menu opens through it. - Thread PR panel renders the watch status line again (#98). - PR page filter counts read every involvement plus the search, ignore the unsearched baseline while a search is typed, show loading rows, and the applied filters render as removable pills below the controls. - Pinned sidebar rows keep the pin trailing, outside the hover swap (#150). - Work log labels read/search tools by what they did instead of "linked issues" (#424). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hovering a pinned thread in the sidebar made its pin icon slide left, because the pin sat just before the status slot and the hover-only actions (draft discard, new tab, snooze, Settle) widen that slot.
The pin is now the trailing item of the card row, outside the status/actions swap, so it stays put. Unpinned rows that support pinning also get a Pin thread action in the same hover group, using the existing
attemptPinpath. Slim rows (settled/snoozed tail) already overlay their actions absolutely, so they are unchanged.Before/after screenshots not captured yet.
Done with Claude Opus 5.5 in Claude Code (via T3 Code).
🤖 Generated with Claude Code