Skip to content

add --select compatibility flag #2078

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

ItsZariep
Copy link

@ItsZariep ItsZariep commented Jul 15, 2025

Some scripts rely on the --select flag to highlight files in file managers. While caja, dolphin, and nautilus support this flag, pcmanfm-qt currently does not.

Currently, pcmanfm-qt triggers an argument parser error when using --select, and if not, it open the file with the user selected mime app.


This simple PR adds support for the --select flag in pcmanfm-qt using the already existing Application::ShowItems method.

With this change, --select will be accepted by pcmanfm-qt without errors, improving compatibility with scripts expecting this flag.

@tsujan
Copy link
Member

tsujan commented Jul 15, 2025

I'm afraid this is a duplication of a standard D-Bus method, which is independent of FM:

qdbus6 org.freedesktop.FileManager1 /org/freedesktop/FileManager1 ShowItems 'FILE_PATH' "0"

While caja, dolphin, and nautilus support this flag…

While they support it, it isn't a standard, and we avoid duplication.

BTW, Caja and some other GTK file managers were forked from Nautilus.

@tsujan
Copy link
Member

tsujan commented Jul 15, 2025

… improving compatibility with scripts expecting this flag.

Scripts should be written in such a way that's independent of FM. Well-known FMs follow freedesktop.org standards.

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