Fix an issue that custom query hotkey cannot work with context menu page#4182
Fix an issue that custom query hotkey cannot work with context menu page#4182Jack251970 merged 1 commit intodevfrom
Conversation
Added a call to App.API.BackToQueryResults() after showing the main window when a hotkey is triggered. This prevents issues that can occur if the current page is a context menu before changing the query.
|
Warning Rate limit exceeded@Jack251970 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 9 seconds before requesting another review. ⌛ 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. 📒 Files selected for processing (1)
✨ 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 |
|
🥷 Code experts: no user but you matched threshold 10 Jack251970 has most 👩💻 activity in the files. See details
Activity based on git-commit:
Knowledge based on git-blame: ✨ Comment |
|
Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX. |
There was a problem hiding this comment.
Pull request overview
This PR fixes an issue where custom query hotkeys could cause problems when triggered while viewing a context menu page. The fix ensures that the application returns to the query results page before changing the query.
- Added a defensive call to
BackToQueryResults()before changing the query in the custom hotkey handler
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…_hotkey Fix an issue that custom query hotkey cannot work with context menu page
This pull request introduces a small but important update to the hotkey handling logic. Now, when a custom query hotkey is triggered, the application will always return to the query results page before changing the query. This ensures that if the user is currently on a context menu page, switching queries will not cause any issues.
HotKeyMapper.cs, added a call toApp.API.BackToQueryResults()before changing the query to prevent issues when the current page is a context menu.