Review Request: [KTextEdit] Handle actions depending on modes/flags
Christoph Feck
christoph at maxiom.de
Thu Nov 3 21:19:37 GMT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102919/
-----------------------------------------------------------
(Updated Nov. 3, 2011, 9:19 p.m.)
Review request for kdelibs.
Changes
-------
Any feedback if this can be commited to KDE/4.7 ?
Description
-------
Changes:
* handle shortcuts for actions that modify the text only when not readOnly()
* handle shortcuts for Find/Replace actions only when findReplaceEnabled()
* show Find actions in context menu even if readOnly()
Unsure if:
* we can actually do this, or if we need to consume the shortcuts
* this goes to KDE/4.7 or frameworks
This addresses bug 284433.
http://bugs.kde.org/show_bug.cgi?id=284433
Diffs
-----
kdeui/widgets/ktextedit.cpp d9877b6
Diff: http://git.reviewboard.kde.org/r/102919/diff/diff
Testing (updated)
-------
I am using this patch since two weeks, and haven't found any regression.
Thanks,
Christoph Feck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20111103/06b1703f/attachment.htm>
More information about the kde-core-devel
mailing list