fix(desktop): rebind preset hotkeys from Cmd+Shift+Number to Ctrl+Number#993
fix(desktop): rebind preset hotkeys from Cmd+Shift+Number to Ctrl+Number#993
Conversation
Cmd+Shift+3/4/5 conflict with macOS screenshot shortcuts. Ctrl+Number is available and follows the familiar Chrome tab-switching pattern.
|
Warning Rate limit exceeded
⌛ 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. 📝 WalkthroughWalkthroughThis PR modifies 9 preset hotkey definitions in the shared hotkeys configuration, changing their key combinations from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
Fix hook call order by using wrapper pattern - outer component checks if hotkeyId exists before rendering inner component that calls hook.
🧹 Preview Cleanup CompleteThe following preview resources have been cleaned up:
Thank you for your contribution! 🎉 |
Summary
Cmd+Shift+1-9toCtrl+1-9Cmd+Shift+3/4/5)Test plan
Ctrl+1-9opens corresponding presetsCmd+Shift+3/4/5) work normallySummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.