kishoreg added a comment.
In D21412#470195 <https://phabricator.kde.org/D21412#470195>, @cullmann wrote: > For the shortcut: I guess you should use KStandardShortcut::ActualSize(). I initially tried using this, but it doesn't compile. It complains that ‘ActualSize’ cannot be used as a function. Just using KStandardShortcut::ActualSize results in the shortcut being set to '8' on my machine. > As we now have three actions just for zooming I think some "Zoom" sub-menu would be nice. Should I make this change here, or in a separate pull request? REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D21412 To: kishoreg, #ktexteditor, cullmann Cc: cullmann, kwrite-devel, kde-frameworks-devel, domson, michaelh, ngraham, bruns, demsking, sars, dhaumann
