GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||
GeezerVison is showing github.com — feat(site/src): show only pinned projects in the Coder Agents sidebar by jaaydenh · Pull Request #32040 · coder/coder Conversation
Contributor
First frontend layer of the Projects page and pinned projects design, on top of the backend in #32025 and #32026. The sidebar now shows only the projects the user has pinned.
Later PRs in this stack move pinned projects into the Pinned area, add drag reordering, and add the move-to-unpinned-project dialog. Verification
Note Most of this PR was written by AI agents (Xum, with UAT run through Coder Agents). The verification above is from those runs.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
The Projects section now lists the user's pinned projects from the project summaries, in pin order, with an unread count on collapsed folders and an Unpin project item in the folder menu. The sidebar chat list passes pinned_projects_only, so chats in unpinned projects leave the sidebar, and a Projects nav item under New chat links to /agents/projects.
Contributor
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 0353cd5f4f
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
…at watch reconnects
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 0a4ea36e1f
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
… needs action A chat that pauses for input publishes only an action_required watch event, so the project summaries now refresh on it too.
… list When a chat is created or unarchived in a project the user has not pinned, the watch handler no longer prepends it to the pinned projects only sidebar lists, which the server would leave it out of.
… the sidebar after unpinning a project Unpinning removes the folder whose menu held focus, so the closing menu now moves focus to the next pinned project folder, else the previous one, else the Projects nav item.
…ct.RefObject for the unpin focus fallback
…e unpin focus helper out of the render scope A closure defined among the component's hooks lost its memoization and failed the React Compiler check, so the neighbor lookup is a module function.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 6269009af3
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
…ts of a project the summaries list as unpinned After an unpin, the summaries can refetch before the chat list, so the project's chats showed under Chats until the list caught up. Grouping now takes every summary and holds back the unpinned chats of a project they list as unpinned. Chats of a project the summaries lack still show.
Unpinning marks the project unpinned in the cached summaries before the request runs, so its folder and chats leave the sidebar at once, and restores the summaries if the request fails.
…e last project on a small screen The Projects nav item hides below the sm breakpoint, so unpinning the last project moved focus to a hidden link and then to the body. Focus now goes to the first rendered of the Projects nav item and the chat filters button.
…er tests to the reorder layer They keep this layer under the review size budget, and the reorder layer adds them back.
|
@codex review |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
At least 1 approving review is required to merge this pull request.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.