https://bugs.kde.org/show_bug.cgi?id=526282

cwo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-desktop/-/commit/
                   |                            |69996be2afc46c87b8ae922e881
                   |                            |b95caf5f7a2ef
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #9 from cwo <[email protected]> ---
Git commit 69996be2afc46c87b8ae922e881b95caf5f7a2ef by Christoph Wolk.
Committed on 31/10/2026 at 16:30.
Pushed by cwo into branch 'master'.

containments/desktop: open menu at currentItem onMenuPressed

Currently, FolderView always opens its context menu at the pointer
position. This makes sense for opening the menu with a right-click, but
the same logic is used when the Menu key is pressed. (If nothing is
selected and we open the containment's context menu instead, the menu
opens at (0,0). This is not how the Menu key works in Dolphin (where it
appears to the bottom right of the current item), somewhat inconvenient
with keyboard navigation (as the pointer position may be far from where
the user is looking) and can be misleading, as it makes it seem like the
menu applies to the file where the context menu is attached to, rather
than the currently selected files.

Instead, we pass in whether the menu should open at the cursor position
or not, and if not, open at the bottom right of the visualParent. This
gives almost identical behavior to Dolphin with regard to where the menu
pops up (with rubberband selection, the two may disagree which is the
currentItem).

M  +1    -1    containments/desktop/package/contents/ui/FolderView.qml
M  +7    -2    containments/desktop/plugins/folder/foldermodel.cpp
M  +1    -1    containments/desktop/plugins/folder/foldermodel.h

https://invent.kde.org/plasma/plasma-desktop/-/commit/69996be2afc46c87b8ae922e881b95caf5f7a2ef

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to