| cullmann |
| KTextEditor |
This change adds a setting to enable or disable the ability to move the selected text by drag-and-drop. It is on by default, which means the default behavior is not changed. Disabling this setting is useful for people like me who generally use Ctrl-X/V for moving text. For those, the drag-and-drop is never intended, but sometimes triggered accidentally, so it is good to be able to remove this pitfall.
FEATURE: 391450
I tested it manually.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Think this is ok, given for some use cases one might want to disable it.
Thanks for taking care!
When you do UI changes, please always add a screenshot. In this case of the changed config page :)
| src/utils/kateconfig.h | ||
|---|---|---|
| 986 | The enum order doesn't matter, for all storage stuff the string keys are used. There shall only be no gaps. | |
Thanks for having a look. Here is a screenshot of the config page with the new option on it (third from the bottom):
Do you think that the label there is okay as-is? Or should it perhaps rather be something like "Enable moving selected text by drag and drop" to be more clear?
I have no issues with the label, matches the other in that category, not sure if "Enable" does make it better or worse.