hein added a comment.
I don't get what this is trying to achieve. 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? Also opening the context menu is supposed to work already (perhaps it regressed ...); 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. 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
