kossebau added inline comments.

INLINE COMMENTS

> spellcheck.cpp:222
> +            Plasma::QueryMatch match(this);
> +            match.setType(Plasma::QueryMatch::HelperMatch);
>              match.setIconName(QStringLiteral("checkbox"));

Not sure if HelperMatch is the correct thing here. Similar runners use 
InformationalMatch here, so on selection the term is copied into the runner.

What is the modern krunner way to support the use-case of someone wanting to 
know the spelling of some word, and then copy the correct or suggested into the 
clipboard for further usage?

REPOSITORY
  R114 Plasma Addons

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

To: kossebau, broulik
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to