Skip to content

Introduce RemoveAction for context menu

Derek Christ requested to merge derek/plasma-desktop:work/context_menu_fix into master

Fixes bug 438930. (Visual glitch when pressing and releasing the shift key) This commit tries to mirror the RemoveAction that Dolphin already has, therefore some of the code in removeaction.cpp is the same as in Dolphin. However, since the folderplugin handles the actions a bit differently than Dolphin, some things differ in the implementations.

Commit message:

The RemoveAction can either represent the delete action or the move to
trash action, depending on the shift modifier state.
This inplace swapping instead of adding and removing both actions fixes
a visual glitch that could be noticed by pressing and releasing the
shift key multiple times.

BUG: 438930
FIXED-IN: 5.22

Merge request reports