<table><tr><td style="">leinir added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25173">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D25173#559588" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25173#559588</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I still see "Loading Data" in a passiveNotification as well as in text under the loading indicator.</p></div>
</blockquote>

<p>i'm seeing a couple of bits which really probably want to be turned into something else (but as separate diffs, as i'm not really keen on this ending up yet another several-months-long saga):</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Loading data (which should really just be an inline item (probably the footer) with a spinner to indicate when the model is fetching more items)</li>
<li class="remarkup-list-item">Loading one preview (which is basically just debug information and should be turned into that)</li>
</ul>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Do we really need the passiveNotifications at all? I think we can probably improve the user experience vy removing them entirely. For example instead of "Loading one preview" we can just show individual loading indicators on the previews themselves</p></blockquote>

<p>i'm afraid we do - as you might gather from that short list above, there's an enormous amount of of these bits of information being thrown back from the engine, and until we're certain we have captured all of them by some other method (and removed the noise), we really can't just get rid of it. (yes, i realise how much you dislike[1] them, and i'm quite happy to move towards removing them, we just can't do it by just straight up removing them entirely. We also can't simply replace it with a dialog box either, because /some/ are informational and can be safely ignored, while some are important, and there's no straightforward way to tell which is which)</p>

<p>[1] was going to say hate, but it's a terribly strong term, and i feel like we probably need to reserve it for things which actually deserve that strength ;)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25173">https://phabricator.kde.org/D25173</a></div></div><br /><div><strong>To: </strong>leinir, KNewStuff, Frameworks, ngraham, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>