Skip to content

🤖 feat: link the footer GitHub slug to the repository - #3762

Merged
ibetitsmike merged 1 commit into
mainfrom
mike/footer-3k0z
Jul 30, 2026
Merged

🤖 feat: link the footer GitHub slug to the repository#3762
ibetitsmike merged 1 commit into
mainfrom
mike/footer-3k0z

Conversation

@ibetitsmike

Copy link
Copy Markdown
Contributor

Summary

The workspace footer already shows the GitHub owner/repo slug once a PR is detected on the branch, but it was inert text. This makes it a link to the repository page.

Background

The slug arrives from PRStatusStore, which parses owner, repo, and number out of the github.com PR URL returned by gh pr view. So whenever the slug renders, we already know the exact repository it names, and the repo URL is unambiguous. The neighbouring PR #N badge is clickable; the slug looked identical in weight but did nothing.

Implementation

FooterRepositoryLabel now renders an <a> to https://github.com/{owner}/{repo} with target="_blank" and rel="noopener noreferrer", which the desktop main process routes through shell.openExternal via setWindowOpenHandler. Styling matches the sibling "Last prompt" pill (h-5, rounded-md, hover:bg-hover) so the affordance reads as interactive, and shrink-0 is retained because the footer row scrolls horizontally.

Behaviour without a detected PR is unchanged: the footer still shows the plain project label. The frontend has no git remote URL today, so the link is scoped to the PR-detected case.

Validation

Checked in Storybook against App/PhoneViewports IPhone16ePRLinkPlacement (which mocks gh pr view): the anchor resolves to https://github.com/coder/mux, carries target="_blank" and rel="noopener noreferrer", and the tooltip renders on hover.

Red-green on the new unit test: removing the href and removing the data-testid each fail only that test.

Risks

Low. One component, no state or IPC changes. Pixel snapshots containing the footer shift by a few pixels from the new padding on the slug.


Generated with mux • Model: anthropic:claude-opus-5 • Thinking: max

The footer already shows owner/repo once a PR is detected on the branch, but it
was inert text. Make it an external link to the repo's GitHub page, styled like
the neighbouring Last prompt control so the affordance is visible.

---

_Generated with `mux` • Model: `anthropic:claude-opus-5` • Thinking: `max`_

<!-- mux-attribution: model=anthropic:claude-opus-5 thinking=max -->
@ibetitsmike

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

Reviewed commit: d1a03a74c9

ℹ️ 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".

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".

@ibetitsmike
ibetitsmike added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@ibetitsmike
ibetitsmike merged commit a9a6b44 into main Jul 30, 2026
20 of 21 checks passed
@ibetitsmike
ibetitsmike deleted the mike/footer-3k0z branch July 30, 2026 14:14
mux-bot Bot added a commit that referenced this pull request Jul 30, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Jul 30, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Aug 1, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Aug 1, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Aug 1, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Aug 2, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
mux-bot Bot added a commit that referenced this pull request Aug 2, 2026
The repository link added in #3762 duplicated the 'Last prompt' pill styling verbatim; extract it so the two footer affordances cannot drift apart.
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