D15929: Add a 'Properties' entry in the Places panel context menu

[Thomas Surrel]

thsurrel added a comment.


  In D15929#336818 <https://phabricator.kde.org/D15929#336818>, @cfeck wrote:
  
  > Maybe I should actually read the code ... I guess what I want is that you move the 'Properties' above the seperator, not after the 'Edit ...' entry.
  
  
  Just to be sure I understand you, you mean like this (for Places):
  
    Properties ...
    Open in Window
    Open in Tab
    -------------------
    (Edit....)
    Hide Entry
    Hide Section
  
  And for Devices:
  
    Unmount
    -------------------
    Properties ...
    Open in Window
    Open in Tab
    Hide Entry
    Hide Section
  
  Or should we also add a separtor between 'Open's and 'Hide's in the Devices case ?
  (and yes, probably better in another patch, since this one is not sure to make its way in anyway)

REPOSITORY
  R318 Dolphin

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

To: thsurrel, #dolphin, #plasma, #vdg
Cc: elvisangelaccio, cfeck, ngraham, abetts, kfm-devel, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181004/beae7802/attachment.htm>