D19950: Move Empty Trash and Safely Remove down in places context menu

[Tigran Gabrielyan]

tigrang created this revision.
tigrang added a reviewer: Dolphin.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
tigrang requested review of this revision.

REVISION SUMMARY
  Bug report only mentions moving Safely Remove down a few rows, however, that makes it inconsistent with Empty Trash.
  This change moves both actions down and adds a separator above.
  
  Before top row, after bottom row
  F6708203: menus.png <https://phabricator.kde.org/F6708203>
  
  Questions:
  
  1. Leave Trash action as it was?
  2. Remove top separator?
  3. Move action above Properties?
  
  BUG: 405620

REPOSITORY
  R318 Dolphin

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

AFFECTED FILES
  src/panels/places/placespanel.cpp

To: tigrang, #dolphin
Cc: kfm-devel, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190321/09956982/attachment.htm>