ngraham added a comment.
In D22514#512182 <https://phabricator.kde.org/D22514#512182>, @aspotashev wrote: > In D22514#502062 <https://phabricator.kde.org/D22514#502062>, @apol wrote: > > > Please review D22540 <https://phabricator.kde.org/D22540> so I can offer a better solution for @broulik's concerns. > > > You already fixed all of those, right? > > IMHO the busy indicator should appear next to the results list, for example right on top of it, because the busy indicator is related to the results, not to the input text. A busy indicator in TextField might be understood as if something is going to happen inside that text field. I would tend to agree. Also now it's driving me slightly crazy that the Plasma busy indicator has a different look from the QQC2-desktop-style one! :) REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D22514 To: apol Cc: 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, mart
