fix: make "view" button in comments feed an anchor link#4721
Conversation
Previously the "view" control was a <button> that called window.open in its onClick handler. As a result, middle-click (mouse-wheel click) did not open a new tab and instead triggered the browser's autoscroll cursor. Render it as a real anchor with target="_blank" so native middle-click and ctrl/cmd-click behavior works. Fixes #4585 Co-authored-by: Cemre Inanc <cemreinanc@users.noreply.github.com>
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Cleanup: Preview Environment RemovedThe preview environment for this PR has been destroyed.
Cleanup triggered by PR close at 2026-05-13T10:13:54Z |
Fixes #4585
Summary
The "view" control in the comments feed was a
<button>that calledwindow.openon click. Because the underlying element was a<button>, mouse-wheel (middle) click triggered the browser's autoscroll cursor instead of opening a new tab.This PR changes the control to a real anchor link (via the shared
Buttoncomponent'shrefprop, which renders as a Next.jsLink) withtarget="_blank", so middle-click and ctrl/cmd-click open a new tab natively.Test plan
Generated with Claude Code