D14199: Simplify the context menu handling functions in FoldersPanel

David Hallas noreply at phabricator.kde.org
Wed Jul 18 17:32:53 BST 2018


hallas added a comment.


  In D14199#294244 <https://phabricator.kde.org/D14199#294244>, @broulik wrote:
  
  > I got completely confused by the code, `TreeViewContextMenu` internally creates a `QMenu` and then does `exec()` in that confusingly named `open()` method. I assumed `TreeViewContextMenu` *was* the menu.
  >
  > Looking good, thanks!
  >  Do you have commit access?
  
  
  No, I don't have commit access. How do I go about getting it?

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D14199

To: hallas, broulik
Cc: broulik, kfm-devel, spoorun, navarromorales, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180718/2061d2ca/attachment.htm>


More information about the kfm-devel mailing list