fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

fix(pull-requests): align checkout control with author - #9196

Merged
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author
Sep 2, 2026
Merged

fix(pull-requests): align checkout control with author#9196
maria-rcks merged 1 commit into
pingdotgg:mainfrom
maria-rcks:t3code/move-pr-tabs-beside-author

Conversation

@maria-rcks

@maria-rcksmaria-rcks commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Moves the pull request checkout command onto the author metadata row, leaving the branch and file stats together below. The command now uses the same plain click-to-copy treatment and inline copied feedback as the branch name.\n\nVerified with the web typecheck, targeted lint, formatting, and live browser checks at 1280×800 and 1024×800.\n\n### Before\n\nBefore: checkout command on the branch and file stats row\n\n### After\n\nAfter: plain checkout command beside the author metadata\n\nGenerated with gpt-5.6-sol in T3 Code through the Codex harness.


Note

Low Risk
UI-only layout and copy affordances in the pull request detail header; no API, auth, or data-path changes.

Overview
Moves the PR checkout command onto the author/“updated” metadata row and keeps the branch line focused on base → head plus file/diff stats.

Introduces PullRequestCopyableCode so branch name and checkout command share the same click-to-copy pattern: truncated monospace label, inline “Copied” swap, tooltip, focus/hover styling, and larger touch targets. The old outline button with CopyIcon on the branch row is removed.

Copy feedback changes: successful checkout copies no longer raise a success toast (inline state only); copy failures still show an error toast via onError.

Reviewed by Cursor Bugbot for commit 1c02c31. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Move checkout control to author row and add PullRequestCopyableCode

  • Adds PullRequestCopyableCode, a reusable copy-to-clipboard control with a 1600ms copied state, configurable labels, optional error callback, and shared hover/focus/coarse-pointer styling
  • Refactors PullRequestDetailPanel.tsx to use PullRequestCopyableCode for both the head branch name and the checkout command, removing inline clipboard hooks and CopyIcon
  • Moves the checkout command from the branch metadata row into the author/update metadata row
  • Behavioral Change: successful checkout-command copies no longer show a success toast; clipboard failures still show an error toast. The checkout control is keyed by its command so copied state resets when the command changes

Macroscope summarized 1c02c31.

@github-actionsgithub-actionsBot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch 2 times, most recently from 2a1367e to c508dccCompareSeptember 2, 2026 08:53
@maria-rcksmaria-rcks changed the title fix(pull-requests): align diff stats with authorfix(pull-requests): align checkout control with authorSep 2, 2026
@github-actionsgithub-actionsBot added size:M 30-99 changed lines (additions + deletions). and removed size:S 10-29 changed lines (additions + deletions). labels Sep 2, 2026
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from c508dcc to ebd3a69CompareSeptember 2, 2026 08:56
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from ebd3a69 to 012a215CompareSeptember 2, 2026 09:03
@github-actionsgithub-actionsBot added size:L 100-499 changed lines (additions + deletions). and removed size:M 30-99 changed lines (additions + deletions). labels Sep 2, 2026
Comment threadapps/web/src/components/pullRequest/PullRequestDetailPanel.tsx Outdated
@macroscopeapp

macroscopeappBot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 1c02c31

Macroscope's review found this PR approvable — This is a contained pull-request header UI refactor that preserves the existing clipboard actions while changing their placement and feedback presentation. Its runtime impact is limited to local rendering and clipboard interaction, with no backend, schema, deployment, or sensitive-area changes.

You can add or adjust custom eligibility rules. Learn more.

@maria-rcks
maria-rcksforce-pushed the t3code/move-pr-tabs-beside-author branch from 012a215 to 1c02c31CompareSeptember 2, 2026 09:14
@maria-rcks
maria-rcks merged commit 57a6660 into pingdotgg:mainSep 2, 2026
27 checks passed
github-actionsBot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 2, 2026
## What's Changed
* fix(server): discover project skills for Claude by @anirudhsama in pingdotgg/t3code#9210
* feat(web): redesign provider editor and models list by @StiensWout in pingdotgg/t3code#8508
* fix(web): prevent connection rows from wrapping during removal by @MatthewFeroz in pingdotgg/t3code#8706
* fix(pull-requests): align checkout control with author by @maria-rcks in pingdotgg/t3code#9196
* fix(release): pin patched expo-sharing version by @willsheldon in pingdotgg/t3code#9250
* chore(mobile): update Expo Sharing to 57.0.17 by @StiensWout in pingdotgg/t3code#9248
**Full Changelog**: pingdotgg/t3code@v0.0.39-nightly.20260902.1257...v0.0.39-nightly.20260902.1260
Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.39-nightly.20260902.1260
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L100-499 changed lines (additions + deletions).vouch:trustedPR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@maria-rcks