| elvisangelaccio | |
| broulik | |
| dfaure |
| Frameworks |
BUG: 383984
Don't switch from editable to breadcrumb mode when htting the Escape key.
Clients already implement their own ways to do this, or turn it off:
Tested in KDE Neon; rebuilt Dolphin against patched KIO. Escape key no longer switches to breadcrumb mode when currently in editable mode.
| No Linters Available |
| No Unit Test Coverage |
The question is: why? I find Escape a natural way to exit editing and never used Ctrl+L. I don't fully understand the bug report. If Ctrl+L should only switch to edit mode, how would you exit again if you remove Escape handling?
| src/filewidgets/kurlnavigator.h | ||
|---|---|---|
| 454 | This class is exported, you cannot remove any methods without breaking ABI. See https://community.kde.org/Policies/Binary_Compatibility_Issues_With_C%2B%2B | |
-1 from me. It's a feature, not a bug. Esc is the standard way to return to previous GUI states, you use this subconsciously without needing to actively think about some other way (at least I do). In addition, Windows Explorer also does it the same way.
Clients already implement their own ways to do this
That's not correct for Ctrl+L you cite: This enters edit mode, but does not close it again. Only F6 works for this. Also, this should be a standard feature and not something each application needs to add (differently).
However there's one thing I'd love to be changed: Currently clicking on a folder to navigate away will close edit mode, but not clicking somewhere else outside of the location bar. This could be added and the checkmark button adding clutter removed or turned into an settings menu (more discoverable than the context menu we have currently). That's how Windows Explorer does it too, and it's logical: click inside the breadcrumb bar to open edit mode, click outside to close it again.
The tricky thing here is how this will interact with situations where the edit mode should always be active (I believe there's a config option for that? What about file dialogs?)
The key point from the bug report is probably this:
If the config option 'editable location bar' is used
This patch disables Esc unconditionally, while it should only do it if the config option is set in Dolphin's or the file dialog's preferences.
If it were done that way, I could agree to the patch.
Abandoning in favor of D16980, which I think makes more sense. In retrospect, you guys were right and this was a stupid idea: the escape key should definitely continue to work like it already does.