<table><tr><td style="">kossebau 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/D7194" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/leinir/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@leinir</a> I now completely gave up on the patch which tried to adapt to the non-shared EntryInternal. As the screenshot loading code and the payload download code  rely on the explicitly shared data to attach the fetched screenshots resp. update the download state, that would need some more redesign of logic to update the central EntryInternal storage and from there any interested viewer on that. Which I do not have the energy to try now. So by what I have seen I would vote for a revert as well. + some kf6 notes as you mentioned.</p>

<p><a href="https://phabricator.kde.org/p/apol/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@apol</a> Was this patch created due to some bug seen, or perhaps only of the kind oh-that-seems-wrong-lets-be-nice-and-prevent-things? If the latter that might be handy :)</p>

<p>Too bad that EntryInternal is exposed at all in the KNS API, while there is a public read-only wrapper to it. Seems the person designing this code initially should have named it EntryInternalReallyIMeanIt, but then descendants still might have just helped themselves as now for whatever they needed to ;)</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/D7194" rel="noreferrer">https://phabricator.kde.org/D7194</a></div></div><br /><div><strong>To: </strong>apol, leinir<br /><strong>Cc: </strong>sitter, kossebau, whiting, mutlaqja, broulik, Frameworks<br /></div>