<table><tr><td style="">kossebau created this revision.<br />kossebau added a reviewer: Kate.<br />Herald added a project: Kate.<br />Herald added a subscriber: kwrite-devel.<br />kossebau requested review of this revision.
</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/D16668">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>While the idea seemed smart to reuse kparts plugins as handler inside<br />
the preview plugin, sadly this results in many clashes for keyboard<br />
shortcuts especially for central ones like "Copy".<br />
Which spoils the idea of a live preview while editing the text document,<br />
as one often runs in unusable shortcuts.</p>

<p>Which is a problem in general with toolviews, which might have actions<br />
with shortcuts conflicting with those as registered by the currently used<br />
document view whose actions are integrated in the global app menu.<br />
KParts plugins in a toolview just escalate this more, given they have<br />
the usual view-centric actions with the respective common shortcuts.</p>

<p>A solution might be to extend KXmlGui to somehow support such situations,<br />
and decide who owns the shortcut by looking at the focus.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>BRANCH</strong><div><div>removepreviewplugin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16668">https://phabricator.kde.org/D16668</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>addons/CMakeLists.txt<br />
addons/preview/CMakeLists.txt<br />
addons/preview/Messages.sh<br />
addons/preview/kpartview.cpp<br />
addons/preview/kpartview.h<br />
addons/preview/ktexteditorpreview.json<br />
addons/preview/ktexteditorpreviewplugin.cpp<br />
addons/preview/ktexteditorpreviewplugin.h<br />
addons/preview/ktexteditorpreviewview.cpp<br />
addons/preview/ktexteditorpreviewview.h<br />
addons/preview/previewwidget.cpp<br />
addons/preview/previewwidget.h</div></div></div><br /><div><strong>To: </strong>kossebau, Kate<br /><strong>Cc: </strong>kwrite-devel, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>