! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
thunar: improved whisker menu search
Status:
RESOLVED: FIXED

Comments

Description Yousuf 'Jay' Philips 2020-04-01 05:39:46 CEST
Created attachment 9662 
patch for thunar

As mentioned bug 10694, we should add keywords to .desktop files to improve search in whisker menu.
Comment 1 alexxcons editbugs 2020-04-02 11:07:04 CEST
Good idea !  ... though your patch fails for me when applied to thunar-master

Would be nice to have keywords as well for thunar-bulk-rename.desktop

Regarding the keywords:

File Manager:
finder --> dont add this .. so far we dont have proper a search widget
attributes --> dont add it ... its too generic. If every single application would add that keyword, nothing at all could be found

Settings:
I would not add all these internals keys.
I e.g. if I search for "date" I would not expect to get the preferences of thunar (or of any application where I can configure a date format) Considering that, I would add the following:

thunar;setting;preference;configure;folder;file;manage;explore;browse;directory;directories

Proposal for bulk renamer:

bulk;rename;thunar;folders;directory;directories;
Comment 2 Yousuf 'Jay' Philips 2020-04-02 21:45:19 CEST
(In reply to alexxcons from comment #1)
> Good idea !  ... though your patch fails for me when applied to thunar-master

No probs, will create a new one.

> Would be nice to have keywords as well for thunar-bulk-rename.desktop

roger.

> finder --> dont add this .. so far we dont have proper a search widget

This is meant as the name of the mac file manager - https://en.wikipedia.org/wiki/Finder_(software)

> attributes --> dont add it ... its too generic. If every single application
> would add that keyword, nothing at all could be found

okay.

> Settings:
> I would not add all these internals keys.
> I e.g. if I search for "date" I would not expect to get the preferences of
> thunar (or of any application where I can configure a date format)

Yes i do agree some keys can be removed.
Search for 'date' now and it will show results with 'update', which i wouldn't expect, but its a basic search. The display of the date in a file manager is something important and being able to configure it is an important feature of a file manager's settings app, so if a user search for 'configure date' or 'date format', settings showing up is useful.

> Considering that, I would add the following:
> 
> thunar;setting;preference;configure;folder;file;manage;explore;browse;
> directory;directories

Repeating so many entries that are already in thunar's keywords here isn't beneficial to search. Here is my counter.

thunar;settings;preferences;configure;thumbnails;file size;date format;shortcuts pane;tree view;tabs;

> Proposal for bulk renamer:
> 
> bulk;rename;thunar;folders;directory;directories;

My suggestion

bulk;renamer;renaming;thunar;files;folders;directory;directories;
Comment 3 Yousuf 'Jay' Philips 2020-04-03 16:00:10 CEST
So I filed bug 16628 as you brought up searching for 'date' and found some weird results and boy was i surprised.
Comment 4 alexxcons editbugs 2020-04-03 21:35:49 CEST
> This is meant as the name of the mac file manager - https://en.wikipedia.org/wiki/Finder_(software)
Ok, I did not know finder. Though I still would not add it  ... It's just a name of a file-manager, we as well dont want to add norton commander, nautilus and dolphin here.

> thunar;settings;preferences;configure;thumbnails;file size;date format;shortcuts pane;tree view;tabs;
ok, fine. lets take a try for it

> bulk;renamer;renaming;thunar;files;folders;directory;directories;
fine fo me


> So I filed bug 16628 as you brought up searching for 'date' and found some weird results and boy was i surprised.
True .. possibly would give more accurate results to only match the name and the keywords (or only match the comment if no keywords are set?)
Comment 5 alexxcons editbugs 2020-04-03 21:36:51 CEST
Same bug for other component: Bug#16618, Bug#16619, Bug#16620, Bug#16621
Comment 6 Yousuf 'Jay' Philips 2020-04-03 22:23:26 CEST
Created attachment 9668 
patch for thunar attempt 2

(In reply to alexxcons from comment #4)
> Ok, I did not know finder. Though I still would not add it  ... It's just a
> name of a file-manager, we as well dont want to add norton commander,
> nautilus and dolphin here.

Added it as its the default on mac, similar to having 'explorer' as its the default on windows. If you still disagree, you can remove it from the patch

> True .. possibly would give more accurate results to only match the name and
> the keywords (or only match the comment if no keywords are set?)

Yes it would be good to optimize the search but keywords should be pulled from all three (name, comment, keywords) which would then reduce duplication across them.
Comment 7 Git Bot editbugs 2020-04-06 09:56:44 CEST
Yousuf Philips referenced this bugreport in commit 8ee6f6601f1e8b465ee366238aa92ac74aba94e5

Add keywords to .desktop files to thunar (bug #16618)

https://git.xfce.org/xfce/thunar/commit?id=8ee6f6601f1e8b465ee366238aa92ac74aba94e5
Comment 8 alexxcons editbugs 2020-04-06 10:00:29 CEST
Ok, I kept  explorer and finder.  ... pushed to master.
Comment 9 Yousuf 'Jay' Philips 2020-04-11 18:05:50 CEST
can this be cherry-picked into 4.14 like bug 16618 and bug 16621?
Comment 10 alexxcons editbugs 2020-04-12 21:45:13 CEST
Rather not, because these strings would need translation as well for 4.14 ... not sure how smart tranifex is these days. I dont want to punish our translators too much.
On top it's not directly a bugfix, more like a feature.

Bug #16618

Reported by:
Yousuf 'Jay' Philips
Reported on: 2020-04-01
Last modified on: 2020-04-12

People

Assignee:
Xfce Bug Triage
CC List:
1 user

Version

Version:
1.8.11

Attachments

patch for thunar (1.51 KB, patch)
2020-04-01 05:39 CEST , Yousuf 'Jay' Philips
no flags
patch for thunar attempt 2 (1.72 KB, patch)
2020-04-03 22:23 CEST , Yousuf 'Jay' Philips
no flags

Additional information