broulik added inline comments.

INLINE COMMENTS

> RunCommand.qml:105
> +
> +                opacity: queryTimer.stillQuerying ? 1 : 0
> +                running: visible

Easier would be `!queryTimer.running && results.querying`, then you don't need 
any of the custom state tracking

And then just `onQueryStringChanged: queryTimer.restart()`

Also, maybe longer than 200ms?

REPOSITORY
  R120 Plasma Workspace

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

To: apol, #plasma, #vdg, broulik
Cc: mart, davidedmundson, ngraham, aspotashev, broulik, plasma-devel, 
LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, 
ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

Reply via email to