D6917: [Component Chooser] Add ComboBox with browsers

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Jul 26 10:07:16 UTC 2017


broulik added inline comments.

INLINE COMMENTS

> componentchooserbrowser.cpp:85
> +
> +    const auto &browsers = KMimeTypeTrader::self()->query(QStringLiteral("text/html"), QStringLiteral("Application"));;
> +    for (const auto &service : browsers) {

One could also instead use

  KServiceTypeTrader::self()->query(QStringLiteral("Application"), QStringLiteral("'WebBrowser' in Categories"));

to really list only web browsers. (I wouldn't mind either), currently it lists all browsers I have plus Kate and Kwrite

REPOSITORY
  R119 Plasma Desktop

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

To: broulik, #plasma, #vdg, dfaure
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170726/2d1eada3/attachment-0001.html>


More information about the Plasma-devel mailing list