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