D12337: Give the file dialogs a "Sort by" menu button on the toolbar

[Nathaniel Graham]

ngraham added inline comments.

INLINE COMMENTS

> kdiroperator.cpp:1887
>      KActionMenu *sortMenu = new KActionMenu(i18n("Sorting"), this);
> +    sortMenu->setIcon(QIcon::fromTheme(QStringLiteral("itemize")));
> +    sortMenu->setDelayed(false);

Before the patch lands, this icon will be changed to whatever https://bugs.kde.org/show_bug.cgi?id=393318 yields.

REPOSITORY
  R241 KIO

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

To: ngraham, #frameworks, #dolphin, #vdg, rkflx
Cc: andreaska, markg, broulik, anemeth, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180502/cc4a1d40/attachment.html>