<table><tr><td style="">kossebau edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-hqz6ce7gildubza/" rel="noreferrer">(Show Details)</a><br />kossebau edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-m5a5mrljlfg4qw7/" rel="noreferrer">(Show Details)</a>
</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/D7580" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">The KTextEditor <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">p</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">P</span>review plugin<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">*</span> repeatedly feeds new<div style="padding: 8px 0;">...</div>the stream API of the kpart is used if available.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
*https://frinring.wordpress.com/2017/08/21/look-what-you-have-donewwdo/</span><br />
<br />
This patch adds support for the stream API.<div style="padding: 8px 0;">...</div>and if the same url is load<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ed</span> again, the view state is restored.<div style="padding: 8px 0;">...</div>updated file as e.g. happening with the preview plugin.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
<br />
Open questions:<br />
# use of the streaming API matching intentions?<br />
# Restoring of view state overlaps with support in browser history, for which there is the KParts::OpenUrlArguments property of ReadOnlyPart. That one only supports x/y offset, so how would zoom and other custom state properties be save and retrieved again, if they should?<br />
<br />
This patch should help to solve this questions in general, so it is<br />
known what to do for other kparts.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Editing SVG files in Kate/K<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">d</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">D</span>evelop using th<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">e p</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">e KTextEditor P</span>review plugin<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">*</span> will<div style="padding: 8px 0;">...</div>changes, also is the filesystem no longer used on updates.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
* kde:scratch/kossebau/ktexteditorpreviewplugin</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R383 SVGPart</div></div></div><br /><div><strong>BRANCH</strong><div><div>supportstreamandreload</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7580" rel="noreferrer">https://phabricator.kde.org/D7580</a></div></div><br /><div><strong>To: </strong>kossebau, Frameworks, dfaure<br /></div>