Skip to content

fix(web): the base branch stays readable in the pull request header - #24

Merged
yordis merged 2 commits into
mainfrom
yordis/fix-pr-base-branch-chip
Aug 20, 2026
Merged

fix(web): the base branch stays readable in the pull request header#24
yordis merged 2 commits into
mainfrom
yordis/fix-pr-base-branch-chip

Conversation

@yordis

@yordis yordis commented Aug 20, 2026

Copy link
Copy Markdown
Member
  • The header's job in one line is to say what is being merged into what. The base branch was collapsing to a letter and an ellipsis while the head kept a wide box, so the header dropped the branch a reader opened the review to check and kept the one they were already on.
  • It only showed at narrower widths, which is where most reviews are read in this app: the panel docked beside a thread.
  • Ledger entry: 0014.

yordis added 2 commits August 20, 2026 02:23
Both branch names competed for the same slack, and the base lost every time:
it collapsed to a letter and an ellipsis while the head kept a wide box, so
the one piece of context that says where the change is going was the piece
that disappeared first.

Signed-off-by: Yordis Prieto <[email protected]>
@cursor

cursor Bot commented Aug 20, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
CSS-only flex/truncation tweaks in the PR header; no auth, data, or API behavior changes.

Overview
Fixes narrow pull request headers so the base branch stays legible when space is tight (e.g. panel docked beside a thread), instead of truncating to a single character while the head branch kept a wide box.

In PullRequestDetailPanel, the base branch labels use shrink-0 and a max-w-32 cap in condensed chrome; the head copy button drops max-w-64 shrink so head truncates first—matching “merge into what” as the primary read.

Adds fork ledger 0014 and indexes it in docs/fork/README.md.

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

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@yordis, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 52 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ba3367f9-4408-409f-a7eb-8e14399919af

📥 Commits

Reviewing files that changed from the base of the PR and between c05617d and ec5a3dc.

📒 Files selected for processing (3)
  • apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx
  • docs/fork/0014-base-branch-stays-readable.md
  • docs/fork/README.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Aug 20, 2026
@yordis
yordis merged commit 50093ca into main Aug 20, 2026
9 of 13 checks passed
@yordis
yordis deleted the yordis/fix-pr-base-branch-chip branch August 20, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR 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