D23757: Clean up viewport and single-folder context menus

[Méven Car]

meven added a comment.


  In D23757#526843 <https://phabricator.kde.org/D23757#526843>, @ndavis wrote:
  
  > In D23757#526812 <https://phabricator.kde.org/D23757#526812>, @ngraham wrote:
  >
  > > Right now it's always in its own single-item group, which looks bad IMO. Open to suggestions.
  >
  >
  > Perhaps it should be grouped with and placed just above Properties? We put "Add to Favorites" at the bottom of the context menu in Kickoff and Properties typically doesn't go with anything else.
  
  
  IMO, Properties should stay the last entry of the list with a separator in all cases : it has nothing to do with anything else and is pretty commonly used, so it deserves the last position in the list that users can get accustomed to.
  So I think we should just add a separator about Properties in the viewport case.
  
  Other than that it looks great.

REPOSITORY
  R318 Dolphin

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

To: ngraham, #vdg, #dolphin
Cc: meven, ndavis, kfm-devel, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190907/c6b90c08/attachment.htm>