D12647: Move the inline preview button into the menu

Nathaniel Graham noreply at phabricator.kde.org
Tue May 1 20:45:25 UTC 2018


ngraham created this revision.
ngraham added reviewers: Frameworks, rkflx.
Restricted Application added a project: Frameworks.
ngraham requested review of this revision.

REVISION SUMMARY
  Once previews are on by default and automatically disable themselves for small icon sizes, there's little reason to have the button to toggle them presented to centrally.
  
  This patch moves it to the menu and removes its icon to prevent it from being the only item in that menu that gets an icon.
  
  Depends on D12328 <https://phabricator.kde.org/D12328>

TEST PLAN
  [image]

REPOSITORY
  R241 KIO

BRANCH
  inline-preview-button-in-menu (branched from master)

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

AFFECTED FILES
  src/filewidgets/kdiroperator.cpp
  src/filewidgets/kfilewidget.cpp

To: ngraham, #frameworks, rkflx
Cc: anemeth, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180501/b6cd0487/attachment.html>


More information about the Kde-frameworks-devel mailing list