D17008: Expose "Toggle Comment" feature through tools menu and default shortcut
Silas Lenz
noreply at phabricator.kde.org
Mon Nov 19 21:46:05 GMT 2018
slenz marked an inline comment as done.
slenz added inline comments.
INLINE COMMENTS
> ngraham wrote in katepart5ui.rc:2
> You need to bump the version to 79 or else the changes won't take effect.
That might explain why the changes only sometimes appeared. Fixed :)
> ngraham wrote in kateview.cpp:543
> Could we also change this string to say "Comment/Uncomment" Rather than "Toggle comment"? I know that's //technically// correct because this is a toggle action, but generally one doesn't associate "toggling" things with text editing.
I think they both make sense, but "Toggle Comment" seems to be the established standard. Three out of my four installed editors used that wording (the last one only used "Comment").
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D17008
To: slenz, #vdg, ngraham
Cc: ngraham, kwrite-devel, kde-frameworks-devel, michaelh, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181119/a1149319/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list