luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-06-02 02:40:07 +02:00
Code Issues Packages Projects Releases Wiki Activity

GDesktopAppInfo: prioritize match_type over match_category when searching apps

Browse Source 
Commit 9e2ad88455 improved app search results by allowing to differentiate
their match_type: prefix match or substring match; while giving more priority
to prefix matches over substring matches, but only when they are in the same
match_category[1].

This was a step forward but, as outlined in #3082, still not enough to get
most relevant results first to the user, because apparently (and for the
specific case of desktop app searching) a prefix match in a lower category
is more relevant to the user than a substring match in a higher category.

So that's what this commit implements, i.e. it makes sure prefix matches
are still preferred over substring matches but this time not only when
in the same category but also across different categories.

[1] Match category is the Desktop file key where the match happened.
    They are shown below from top to lesser priority.
      DESKTOP_KEY_Name
      DESKTOP_KEY_Exec
      DESKTOP_KEY_Keywords
      DESKTOP_KEY_GenericName
      DESKTOP_KEY_X_GNOME_FullName
      DESKTOP_KEY_Comment

Fixes #3082
This commit is contained in:
Nelson Benítez León 2023-09-25 03:29:53 +01:00 committed by Philip Withnall
parent 3c22e2745c
commit 40f567aa2a
3 changed files with 35 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
gio/gdesktopappinfo.c
View File

@ -458,7 +458,9 @@ const gchar desktop_key_match_category[N_DESKTOP_KEYS] = {
typedef enum { typedef enum {
/* Lower numbers have higher priority. /* Lower numbers have higher priority.
* Prefix match should put before substring match. * Prefix match should put before substring match, independently of
* category relevance, i.e. a prefix match in 'Keyword' category will
* come before a substring match in a more relevant category like 'Name'.
*/ */
MATCH_TYPE_PREFIX = 1, MATCH_TYPE_PREFIX = 1,
MATCH_TYPE_SUBSTRING = 2 MATCH_TYPE_SUBSTRING = 2
@ -576,7 +578,10 @@ compare_results (gconstpointer a,
} }
else else
{ {
if (ra->category == rb->category) /* We prioritize prefix matches over category relevance e.g. a prefix match in 'Keyword'
* category is better than a substring match in a more relevance category like 'Name'.
*/
if (ra->match_type != rb->match_type)
return ra->match_type - rb->match_type; return ra->match_type - rb->match_type;
return ra->category - rb->category; return ra->category - rb->category;
@ -590,10 +595,10 @@ compare_categories (gconstpointer a,
const struct search_result *ra = a; const struct search_result *ra = a;
const struct search_result *rb = b; const struct search_result *rb = b;
/* Also compare match types so we can put prefix match in a group while /* We prioritize prefix matches over category relevance e.g. a prefix match in 'Keyword'
* substring match in another group. * category is better than a substring match in a more relevance category like 'Name'.
*/ */
if (ra->category == rb->category) if (ra->match_type != rb->match_type)
return ra->match_type - rb->match_type; return ra->match_type - rb->match_type;
return ra->category - rb->category; return ra->category - rb->category;

27
gio/tests/apps.c
View File

@ -34,12 +34,27 @@ main (int argc, char **argv)
setlocale (LC_ALL, ""); setlocale (LC_ALL, "");
if (argv[1] == NULL || g_str_equal (argv[1], "--help")) if (argv[1] == NULL || g_str_equal (argv[1], "--help"))
g_print ("Usage:\n apps --help\n apps COMMAND [COMMAND_OPTIONS]\n\nCOMMANDS:\n list\n " g_print ("Usage:\n"
"search [--should-show-only] TEXT_TO_SEARCH\n implementations INTERFACE_NAME\n " " apps --help\n"
"show-info DESKTOP_FILE\n default-for-type MIME_TYPE\n recommended-for-type MIME_TYPE\n " " apps COMMAND [COMMAND_OPTIONS]\n"
"all-for-type MIME_TYPE\n fallback-for-type MIME_TYPE\n should-show DESKTOP_FILE\n " "\n"
"monitor\n\nExamples:\n apps search --should-show-only ter\n" "COMMANDS:\n"
" apps show-info org.gnome.Nautilus.desktop\n apps default-for-type image/png\n\n"); " list\n"
" search [--should-show-only] TEXT_TO_SEARCH\n"
" implementations INTERFACE_NAME\n"
" show-info DESKTOP_FILE\n"
" default-for-type MIME_TYPE\n"
" recommended-for-type MIME_TYPE\n"
" all-for-type MIME_TYPE\n"
" fallback-for-type MIME_TYPE\n"
" should-show DESKTOP_FILE\n"
" monitor\n"
"\n"
"Examples:\n"
" apps search --should-show-only ter\n"
" apps show-info org.gnome.Nautilus.desktop\n"
" apps default-for-type image/png\n"
"\n");
else if (g_str_equal (argv[1], "list")) else if (g_str_equal (argv[1], "list"))
{ {
GList *all, *i; GList *all, *i;

7
gio/tests/desktop-app-info.c
View File

@ -888,11 +888,12 @@ test_search (void)
"gedit.desktop\n", TRUE, TRUE, NULL, NULL); "gedit.desktop\n", TRUE, TRUE, NULL, NULL);
/* "con" will match "connect" and "contacts" on name with prefix match in /* "con" will match "connect" and "contacts" on name with prefix match in
* first group, then match "Dconf Editor" and "Desktop Icons" with substring * first group, then second group is a Keyword prefix match for "configuration" in dconf-editor.desktop
* match in next group. * and third group is a substring match for "Desktop Icons" in Name of nautilus-classic.desktop.
*/ */
assert_search ("con", "gnome-contacts.desktop nautilus-connect-server.desktop\n" assert_search ("con", "gnome-contacts.desktop nautilus-connect-server.desktop\n"
"dconf-editor.desktop nautilus-classic.desktop\n", TRUE, TRUE, NULL, NULL); "dconf-editor.desktop\n"
"nautilus-classic.desktop\n", TRUE, TRUE, NULL, NULL);
/* "gnome" will match "eye of gnome" from the user's directory, plus /* "gnome" will match "eye of gnome" from the user's directory, plus
* matching "GNOME Clocks" X-GNOME-FullName. It's only a comment on * matching "GNOME Clocks" X-GNOME-FullName. It's only a comment on