D4961: [Folder View] Allow querying currentItem

[Kai Uwe Broulik]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170306/910c5663/attachment.html>