Fix home page result context menus

[DavidGBrett]

Fixes two bugs related to home page results and context menus.

Fix 1 Arrow Input Broken

Fixes #4599 where the right arrow input to open the context menu wasn't working.

Fix 2 Stale Context Menus

Also fixes another bug where the context menu itself was stale and not rebuilt when a new result had its context menu opened.
This was becasue of the ignored query text flag incorrectly applying because each context menu used the same empty string query.
Fixed that by making that check only apply to search queries.

(the problem for fix 2 wasn't obvious before because the plugin indicator plugin for example had nothing to show in its context menu anyways)

---

Summary by cubic

Fixes two home page context menu bugs: the Right arrow now opens menus on results without a search query, and menus rebuild so they no longer show stale options.

• Summary of changes

  • Changed: Right arrow key handling no longer requires a non-empty query to open the context menu.
  • Changed: _ignoredQueryText now applies only when search results are selected, preventing stale context menus/history.
  • Added: Always rebuild context menu/history when their view is active; skip only redundant searches on programmatic query restores.
  • Removed: The empty-query check blocking Right arrow; global use of _ignoredQueryText outside search views.
  • Memory impact: None expected.
  • Security risks: None.
  • Unit tests: No new unit tests; verified via manual testing.

• Release Note

  • Context menus on the home page now open with the right arrow and show up-to-date options.

^{Written for commit 3c5eaaf. Summary will update on new commits.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The Right Arrow handler now opens the result context menu for selected results with an empty query. _ignoredQueryText filtering now applies only during query-results mode, preventing it from suppressing context-menu and history queries.

Changes

Query context-menu flow

Layer / File(s)	Summary
Query state and ignored text handling Flow.Launcher/ViewModel/MainViewModel.cs	SelectedResults clears QueryText when leaving query results. _ignoredQueryText matching now runs only for query-results queries.
Empty-query context-menu shortcut Flow.Launcher/MainWindow.xaml.cs	The Right Arrow handler no longer requires non-empty query text before opening the result context menu.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• Flow-Launcher/Flow.Launcher#3240: Both changes adjust MainViewModel query and context-menu state handling.
• Flow-Launcher/Flow.Launcher#3503: This change modifies the _ignoredQueryText handling introduced by that PR.

Suggested reviewers: jack251970, jjw24

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	The stale context-menu fix is not covered by linked issue #4599, which focuses only on right-arrow navigation.	Link an issue that covers stale context-menu rebuilding, or split that fix into a separate pull request.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes satisfy issue #4599 by enabling the right-arrow key to open home result context menus.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly summarizes the two fixes for home page result context menus.
Description check	✅ Passed	The description directly explains both context-menu bugs and the implemented fixes.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-home-page-result-context-menus

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[cubic-dev-ai]

No issues found across 2 files

_{Re-trigger cubic}

[Jack251970]

LGTM! Thanks for your contribution!