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

Reply via email to