D16668: Remove document preview plugin: current technology harms workflow
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Sun Nov 4 18:48:48 GMT 2018
kossebau created this revision.
kossebau added a reviewer: Kate.
Herald added a project: Kate.
Herald added a subscriber: kwrite-devel.
kossebau requested review of this revision.
REVISION SUMMARY
While the idea seemed smart to reuse kparts plugins as handler inside
the preview plugin, sadly this results in many clashes for keyboard
shortcuts especially for central ones like "Copy".
Which spoils the idea of a live preview while editing the text document,
as one often runs in unusable shortcuts.
Which is a problem in general with toolviews, which might have actions
with shortcuts conflicting with those as registered by the currently used
document view whose actions are integrated in the global app menu.
KParts plugins in a toolview just escalate this more, given they have
the usual view-centric actions with the respective common shortcuts.
A solution might be to extend KXmlGui to somehow support such situations,
and decide who owns the shortcut by looking at the focus.
REPOSITORY
R40 Kate
BRANCH
removepreviewplugin
REVISION DETAIL
https://phabricator.kde.org/D16668
AFFECTED FILES
addons/CMakeLists.txt
addons/preview/CMakeLists.txt
addons/preview/Messages.sh
addons/preview/kpartview.cpp
addons/preview/kpartview.h
addons/preview/ktexteditorpreview.json
addons/preview/ktexteditorpreviewplugin.cpp
addons/preview/ktexteditorpreviewplugin.h
addons/preview/ktexteditorpreviewview.cpp
addons/preview/ktexteditorpreviewview.h
addons/preview/previewwidget.cpp
addons/preview/previewwidget.h
To: kossebau, #kate
Cc: kwrite-devel, michaelh, ngraham, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181104/26615db8/attachment-0001.html>
More information about the KWrite-Devel
mailing list