Summary: | Context menu in an empty directory does not contain "Open with..." | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | Oto Šťáva <oto.stava> |
Component: | general | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | elvis.angelaccio, kfm-devel, nate, oto.stava, vibs.35 |
Priority: | NOR | ||
Version: | 20.08.1 | ||
Target Milestone: | --- | ||
Platform: | Manjaro | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/system/dolphin/commit/b343fdffc52ff42d83afa02f6eaf1f5a359a49a9 | Version Fixed In: | 20.12 |
Description
Oto Šťáva
2020-10-12 07:18:13 UTC
This is currently intentional. We got complaints about showing the menu item for empty folders in the past so we changed it in https://invent.kde.org/system/dolphin/-/commit/14f0cd52f61de7539e2d9c751966594708ed0281. Elvis, what do you think? Should we revert that, or close this as INTENTIONAL? " Any app that registers itself as able to open directories generally can't do anything useful with an empty directory. So, don't show the {nav Open With} menu items for them " looks like above reasoning mentioned in the commit is proved false according to OP's use case. Reverting the commit would be recommended IMO Yeah we should probably revert it, since there is a valid use case that is prevented from that commit. All right, will do. Git commit b343fdffc52ff42d83afa02f6eaf1f5a359a49a9 by Nate Graham. Committed on 13/10/2020 at 00:16. Pushed by ngraham into branch 'master'. Show "Open With" menu items even for empty dirs We found some legitimate use cases for this after all! :) This reverts 14f0cd52f61de7539e2d9c751966594708ed0281 FIXED-IN: 5.12 M +3 -5 src/dolphincontextmenu.cpp https://invent.kde.org/system/dolphin/commit/b343fdffc52ff42d83afa02f6eaf1f5a359a49a9 |