D11324: Select files instead of opening them when trying to open a QUrl as a directory when in fact it is a file.
Emirald Mateli
noreply at phabricator.kde.org
Wed Mar 14 12:07:19 GMT 2018
emateli created this revision.
emateli added reviewers: rkflx, ngraham, dfaure, elvisangelaccio.
Restricted Application added a subscriber: Dolphin.
emateli requested review of this revision.
REVISION SUMMARY
When dolphin is executed with a parameter which happens to be a file, it will open a non existing directory and execute the file. This
patch changes the behaviour to instead select the file and open its parent directory.
This can occurr if you intentionally launch Dolphin with a file name as a parameter (without specifying `--select`), or drag and dropping a
file to its task bar icon. This change is true for all files except archives, since due to a prior
patch(https://git.reviewboard.kde.org/r/119877/) all archives will be openened in a new tab.
Given that, I also propose a second change: To browse archives as a directory only when the appropriate setting("Open Archives as folders")
is enabled.
TEST PLAN
- Launch dolphin with a file as parameter or by dropping an icon to its task icon.
REPOSITORY
R318 Dolphin
BRANCH
select-open-file
REVISION DETAIL
https://phabricator.kde.org/D11324
AFFECTED FILES
src/dolphinviewcontainer.cpp
To: emateli, rkflx, ngraham, dfaure, elvisangelaccio
Cc: #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180314/f832b5c9/attachment.htm>
More information about the kfm-devel
mailing list