D16767: Add option to toggle all hidden panel item to view menu

Elvis Angelaccio noreply at phabricator.kde.org
Sun Nov 11 16:39:34 GMT 2018


elvisangelaccio added inline comments.

INLINE COMMENTS

> ngraham wrote in dolphinmainwindow.cpp:1351
> We don't make menu items conditionally disappear. Please only disable it when it can't be used, don't hide it.

Well, we already do for the action in the places panel context menu (same in the KIO one). Shouldn't we be consistent with that?

REPOSITORY
  R318 Dolphin

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

To: rizzitello, #dolphin, #vdg, elvisangelaccio, ngraham
Cc: cfeck, elvisangelaccio, veqz, davidc, ngraham, kfm-devel, alexde, sourabhboss, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181111/b6748f7b/attachment.htm>


More information about the kfm-devel mailing list