D11569: Improve consistency of "Open With" UI

[Anthony Fieroni]

anthonyfieroni added inline comments.

INLINE COMMENTS

> kfileitemactions.cpp:654
> +            // Always show the top app inline
> +            QAction *act = d->createAppAction(offers.first(), menu);
> +            menu->addAction(act);

Use offers.takeFirst(), it will get and remove.

> kfileitemactions.cpp:656
> +            menu->addAction(act);
> +            offers.removeFirst();
> +

Remove.

REPOSITORY
  R241 KIO

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

To: ngraham, rkflx, #vdg, #frameworks, #dolphin
Cc: anthonyfieroni, abetts, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180322/3afee09b/attachment.htm>