dolphin/src/dolphinviewcontainer.cpp, branch masterPatched KDE Dolphin with Pixel Scaling
https://fiftyfourth.xyz/git/dolphin/atom?h=master2026-03-06T09:50:02Zdolphinviewcontainer: Fix infinite loop in isTopMostParentFolderWritable2026-03-06T09:50:02ZEgor Maksimovundeffox@gmail.com2026-03-06T09:50:02Zurn:sha1:837034490b546beeb7d9a188134a2afb6f2d51b2
Certain urls lead to an infinite loop that causes dolphin to halt. Any url that starts with `file:` and the path doesn't have first `/` will lead to this. For example: `file:test` or `file:another/test`. If dolphin somehow saves this state, the program halt right at the start.
Rename the function to make its usage less confusing.
dolphinviewcontainer: let the user open a file that was a directory2026-03-02T14:33:56ZHarald Sittersitter@kde.org2026-02-10T12:00:28Zurn:sha1:44a1202682277a112588ab2c4b77ce8c31962590
when trying to open a file as directory we run into a IsFile error and
instead open the parent directory. this is a bit inconvenient when the
user actually wants to open a file. allow them to redirect the request
to the correct API instead. this ensures backwards compatible behavior
but introduces new strings that require localization.
ultimately being able to open files this way is a bit of any-feature
though.
CCBUG: 516830
dolphinviewcontainer: do not implicitly open directories as files2026-03-02T14:33:56ZHarald Sittersitter@kde.org2026-02-10T11:55:11Zurn:sha1:4a4da5f73899cdd2a1ef111194e79a620eed7716
we already have xdg-run, we don't need another runner interface ;)
notably if the user disabled the ask-before-execute guard rail from KIO,
they would also implicitly allow other applications to execute any file
by invoking it through dolphin, that would be a bit silly and isn't the
point of the guard rail opt-out.
when detecting a file we now instead open the parent
Should improve
CCBUG: 516830
clang-tidy: avoid copy, use const references2026-01-22T15:27:11ZMéven Carmeven@kde.org2026-01-22T15:15:09Zurn:sha1:1acd0dfe09a34817d8daf58db2c53b2262eec7b7clang-tidy: use default for trivial constructor2026-01-22T15:14:16ZMéven Carmeven@kde.org2026-01-22T15:04:09Zurn:sha1:174bc684603326de15e933b3bb24ffb739f3291eOn Middle-click, allow to open scripts with the second app2026-01-14T13:59:55ZMéven Carmeven@kde.org2026-01-14T13:58:00Zurn:sha1:31f7a17a78cab2571a8f53710e82f2c472e6b2d5
When holding shift when middle-clicking.
Amends: a25f2cc67cbab7f001c883931a433f1841223a87
CCBUG: 496687
Use Q_ASSERT instead of Q_CHECK_PTR for plain checks of pointer value2026-01-10T15:14:26ZFriedrich W. H. Kossebaukossebau@kde.org2026-01-09T15:16:48Zurn:sha1:5fff1e414a060c0619a97ad8d3f5665e3f5a4878
Q_CHECK_PTR (other than the name suggests) is intended only to check the
success of memory allocations.
Using it for any pointers instead can be misleading due to the
"Out of memory" log printed in case of hits.
Revert "Avoid implicitly selecting items"2025-11-12T14:54:30ZNate Grahamnate@kde.org2025-11-11T17:47:26Zurn:sha1:7419cd326902c64b69802ab3f01656076d3c7a97
This reverts commit 122fee5625f0285ec4ebda79162c72390989eb2a.
This behavior change was well-intentioned, but has significant usability
and speed drawbacks that have not been addressed:
- Keyboard-driven folder manipulation became slower
- Unexpected behavioral differences when opening files with the pointer
compared to when opening them with the keyboard introduced
inconsistency and cognitive load
- Unexpected opening of selection mode during fast operation introduced
the potential for confusion and additional errors
- Dolphin's behavior became inconsistent with that of other file
managers users may be accustomed to
The bug tracker, discuss.kde.org, and Reddit are full of complaints
about this change. It's been a year now; I think it's clear that many
Dolphin users have not gotten used to and accepted it. I have to count
myself as one of them, I'm afraid. I've tried to get used to it for a
year, but I just have not been able to.
I don't believe the benefits of this change outweigh the drawbacks, so
let's revert it.
24d859cf19e90fa22ed687b36a68231625c1bd80 was explicitly mentioned as a
thing that should also be reverted in this case, but it's already been
done.
BUG: 494125
BUG: 511966
CCBUG: 424723
CCBUG: 492404
FIXED-IN: 25.12.0
dolphinview: On shift+middle-click, open file with third app for the open with list2025-11-04T09:48:43ZTomasz Kottomasz.kot13@gmail.com2025-11-04T09:48:43Zurn:sha1:4e89d3b88aabd371ef0b0b10c6a5fc95c8765250
Added the feature to open the selected file with the third application from the associated applications list on shift and middle-click. Generalized the middle-click slot to open the file with the app on a specified index, since the behavior should be almost the same for those two actions.
Similar to: d27ee07de7558470ef7b497fbe3d3504ce7cad07
BUG: 510237
GeneralViewSettingsPage: Improve the wording "Browse archives as folders"2025-10-30T14:38:29Zyu shuoqiyusq77@gmail.com2025-10-30T14:38:29Zurn:sha1:3dda2b703f03212d3c90597b01ba31a69db39941
Replace the wording "Open archives as folder" to the wording "Browse archives as folders".
BUG: 499782