Skip to content

Add new 'Search' option to main menu#2258

Closed
RushiChaganti wants to merge 1 commit into
basecamp:masterfrom
RushiChaganti:master
Closed

Add new 'Search' option to main menu#2258
RushiChaganti wants to merge 1 commit into
basecamp:masterfrom
RushiChaganti:master

Conversation

@RushiChaganti
Copy link
Copy Markdown

@RushiChaganti RushiChaganti commented Oct 6, 2025

This PR adds a Search option to the Omarchy main menu, allowing users to quickly find and execute any menu action. The search interface is dmenu-based and integrates themes, fonts, and power profiles for faster navigation.

Note: Another contributor, KoushikScripts
, was also working on a similar fuzzy search feature. Their approach is slightly different, but both aim to improve menu navigation.

Key Changes:

  • Added extract_menu_actions() to list all menu items with their commands.
  • Added show_search_menu() for searchable menu selection.
  • Integrated dynamic themes, fonts, and power profiles.
  • Updated go_to_menu() and show_main_menu() to include the new Search option.

Benefit:

Users can quickly access any menu item without navigating multiple sub-menus.

Main Menu image

Search Menu image

@dhh
Copy link
Copy Markdown
Member

dhh commented Oct 6, 2025

Appreciate the suggestion, but I don't love the dynamics of that. If we're going to have a menu-wide search, it has to be fully integrated into the walker setup.

@dhh dhh closed this Oct 6, 2025
@RushiChaganti
Copy link
Copy Markdown
Author

Hey @dhh ,
Just to confirm — you’d like me to integrate this throughout the system, including the walker menus for all apps, and combine everything into a single unified search option, correct?

@dhh
Copy link
Copy Markdown
Member

dhh commented Oct 7, 2025

I don't know if this is currently feasible. What I'd prefer is that there's one input field, and that it by default just filters the visible options we have, but, maybe, if you hold down a modifier, like alt, it'll search the global space. But I don't want this to be a big, invasive change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants