Skip to content

fix(launcher): DankLauncherV2 was missing id and genericName search - #3122

Open
z1-0 wants to merge 1 commit into
AvengeMedia:masterfrom
z1-0:feat/search-by-exec
Open

fix(launcher): DankLauncherV2 was missing id and genericName search#3122
z1-0 wants to merge 1 commit into
AvengeMedia:masterfrom
z1-0:feat/search-by-exec

Conversation

@z1-0

@z1-0 z1-0 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

What this fixes

Almost every mainstream launcher matches by id, and most also support generic name. DankLauncherV2 didn't have it.

AppSearchService already matches apps by their desktop id and generic name, but DankLauncherV2's Scorer never picked those up. So when an app only matched on id or genericName, launcher just skipped it.

What changed

  • Match GenericName (prefix 800, contains 400).
  • Match the desktop id (350), after dropping the .desktop suffix.
  • These only run once name, subtitle, and keyword matches fail, so existing search results stay the same.

Tested

I tested it on my own machine. Typing an app's id (org.kde.dolphin) or its generic name (Web Browser) now finds it in the V2 launcher.

@z1-0
z1-0 marked this pull request as draft August 20, 2026 10:04
@z1-0
z1-0 force-pushed the feat/search-by-exec branch 2 times, most recently from 3d31382 to 3cb868c Compare August 20, 2026 10:57
…rV2 scorer

Scorer was dropping apps that only matched on id or genericName, even though AppSearchService.searchApplications already matches both. Add id (350) and genericName (prefix 800, contains 400) tiers so the V2 launcher ranks those apps the same way.
@z1-0
z1-0 force-pushed the feat/search-by-exec branch from 706364a to 48825b0 Compare August 22, 2026 21:12
@z1-0 z1-0 changed the title feat(launcher): match apps by Exec command name fix(launcher): match apps by id and genericName in DankLauncherV2 scorer Aug 22, 2026
@z1-0 z1-0 changed the title fix(launcher): match apps by id and genericName in DankLauncherV2 scorer fix(launcher): DankLauncherV2 was missing id and genericName search Aug 22, 2026
@z1-0
z1-0 marked this pull request as ready for review August 22, 2026 21:21
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.

1 participant