broulik added a comment.
> Setting the item that has GridView.isCurrentItem to gridView.currentItem should be a no-op. It's already the current item, how can it be more current-item-y? Note that GridView does *not* have a `currentItem` property – I added it in this patch. > Also opening the context menu is supposed to work already (perhaps it regressed ...) It just returns because of that TODO :) > the problem that's marked by a TODO in the code is that it won't open at the right position because FolderModel::openContextMenu uses QCursor::pos() and the mouse pointer might be somewhere else than the item that's been nav'ed to by keyboard. Yup, that's what I need this patch for, so I know the current item so I can show the menu next to it, cf. also https://phabricator.kde.org/D4962 which makes use of this. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4961 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
