D23757: Clean up viewport and single-folder context menus and hamburger menu

[Elvis Angelaccio]

elvisangelaccio added a comment.


  In D23757#528215 <https://phabricator.kde.org/D23757#528215>, @ngraham wrote:
  
  > In D23757#528212 <https://phabricator.kde.org/D23757#528212>, @elvisangelaccio wrote:
  >
  > > While I agree that New Tab/New Window shouldn't be reachable only from the context menu, I'm a bit worried about the height of the control menu. With this patch applied, it is taking almost all the available height on my 1366x768 laptop screen.
  > >
  > > Is there maybe some action that we could remove from the control menu? Maybe "Redisplay" ?
  >
  >
  > I agree with you and I'm all ears regarding things we could remove. Redisplay is definitely a prime candidate IMO.
  >
  > I noticed recently in the ElementaryOS file manager that the refresh button is integrated into the URL navigator, on the right side. I wonder if we could do that too. That would make it still reachable (more so, in fact) despite not being in the default toolbar or the hamburger menu.
  
  
  Yeah. Also "Redisplay" only makes sense on remote URLs or systems where we can't rely on inotify.

REPOSITORY
  R318 Dolphin

BRANCH
  clean-up-context-menu (branched from master)

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

To: ngraham, #vdg, #dolphin, meven
Cc: elvisangelaccio, 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/20190910/af238447/attachment.htm>