GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — feat(site): count every PR in the chats sidebar by DanielleMaywood · Pull Request #29832 · coder/coder

Conversation

@DanielleMaywood

@DanielleMaywood DanielleMaywood commented Sep 24, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Part 6 of 6. Depends on #29831. Supersedes #29547, which merged with a flattened head by mistake.

The chats sidebar and search results now count every PR in diff_statuses, not only the primary ref. A chat with one PR shows that PR's state icon and line stats, even when the primary ref is a newer branch without a PR. A chat with several PRs shows a PR count instead of a single state icon, with no line stats. Screen readers hear "N pull requests".

In the sidebar tree, the kebab menu and the right-click menu of a chat list its PRs in a group between "Copy" and "Archive agent". One PR is a direct link. Several PRs open an "N PRs" submenu. The entries use the same rows as the top bar PR menu. Viewers of a shared chat also see the links. PRs without a URL are not listed.

These menus replace the hover tooltip from earlier commits. The tooltip did not open on touch and had no links to the PRs.

Generated by Coder Agents.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
chatgpt-codex-connector[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@DanielleMaywood
DanielleMaywood marked this pull request as draft September 24, 2026 08:35
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 24, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-24T08:41:19.710165Z 4c65507 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@DanielleMaywood DanielleMaywood added the experimental Changes that might not necessarily be merged, until its approved to proceed with. label Sep 24, 2026
@github-actions github-actions Bot assigned DanielleMaywood Sep 24, 2026
@DanielleMaywood
DanielleMaywood added this pull request to stack #29833 September 24, 2026 08:37
@DanielleMaywood
DanielleMaywood marked this pull request as ready for review September 24, 2026 08:37
@DanielleMaywood DanielleMaywood mentioned this pull request Sep 24, 2026
feat(site): count every PR in the chats sidebar #29547
Merged
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from 4c65507 to ca48391 Compare September 24, 2026 12:03
@DanielleMaywood DanielleMaywood closed this Sep 24, 2026
@DanielleMaywood DanielleMaywood reopened this Sep 24, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 24, 2026
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch 3 times, most recently from ce0f0a4 to ac79137 Compare September 25, 2026 08:59
@DanielleMaywood
DanielleMaywood requested a review from a team as a code owner September 25, 2026 08:59
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from ac79137 to daf7c47 Compare September 25, 2026 09:15
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from daf7c47 to ef08cf5 Compare September 28, 2026 12:11
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from ef08cf5 to 119a611 Compare September 28, 2026 16:30
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from 119a611 to 161c5b0 Compare September 29, 2026 10:13
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch 2 times, most recently from a1b523b to ddc10c9 Compare September 30, 2026 14:37
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch 2 times, most recently from 95eb7d6 to 5c6263c Compare October 1, 2026 09:59
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from 5c6263c to 6a82108 Compare October 5, 2026 09:06
jakehwll
jakehwll approved these changes Oct 5, 2026
View reviewed changes
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch 2 times, most recently from 842ed97 to fd89504 Compare October 6, 2026 08:28
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from fd89504 to ce180fd Compare October 6, 2026 10:14
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from ce180fd to b3f6a64 Compare October 9, 2026 09:36
DanielleMaywood added 8 commits October 9, 2026 09:59
@DanielleMaywood
feat(site): count every PR in the chats sidebar
93c023a 
The sidebar and the search results now follow the full PR list. A
chat with one PR shows that PR's state icon and line stats. A chat
with several PRs shows the PR count instead; one state icon cannot
represent the others. Hovering the icon lists every PR, one row per
PR, each with its state, number, and title.
@DanielleMaywood
refactor(site): name mock data consistently in sidebar stories
be31b5a 
Every mock data fixture now starts with mock: mockChat2DiffStatus,
mockOverflowDiffStatus, mockMultiPRChat, and mockDashboardValue.
@DanielleMaywood
refactor(site): name the popover rows for what they show
d28f816 
The label and PR number derivations stay in the row where they
are used, named prNumber and label to match the top bar.
@DanielleMaywood
style(site): give the PR row derivations room to breathe
3667d9f 
The sole-PR stats and the popover row labels ran as unbroken
blocks of declarations. Each derivation group now sits under its
own comment with a blank line between groups.
@DanielleMaywood
style(site): trim the sidebar comments to their arguments
da9568a
@DanielleMaywood
fix(site): open the multi-PR tooltip from the chat link
c427e46
@DanielleMaywood
fix(site): announce each tooltip PR state to screen readers
fac51d7
@DanielleMaywood
feat(site): list a chat's PRs in its actions menu
edfa69a 
The sidebar tooltip that listed a chat's PRs could not hold links, grew without limit, and opened on every hover of the row. The PRs now sit in the chat's kebab and right-click menus as links, in a group above "Archive agent". One PR is a direct item. Several PRs sit in an "N PRs" submenu that scrolls. Viewers of a shared chat also see the links. PRs without a URL are not listed.
@DanielleMaywood
DanielleMaywood force-pushed the feat/chat-diff-ref-ui branch from b3f6a64 to edfa69a Compare October 9, 2026 10:16
@DanielleMaywood
DanielleMaywood merged commit ff3fa22 into main Oct 9, 2026
34 of 54 checks passed

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jakehwll jakehwll jakehwll approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@DanielleMaywood DanielleMaywood

Labels

experimental Changes that might not necessarily be merged, until its approved to proceed with.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@DanielleMaywood @jakehwll
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.