D21412: Add action to reset font size

2019-05-26 Thread Kishore Gopalakrishnan
kishoreg added a comment. I don't think I have the permissions for that. Could you please do it for me? Name: Kishore Gopalakrishnan Email: kishor...@gmail.com Also, thanks for helping me with this! REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D

D21412: Add action to reset font size

2019-05-26 Thread Kishore Gopalakrishnan
kishoreg updated this revision to Diff 58680. kishoreg added a comment. I've made the suggested changes (use standard reset zoom shortcut, and move zoom option to a submenu). Everything seems to work now. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D2

D21412: Add action to reset font size

2019-05-26 Thread Kishore Gopalakrishnan
kishoreg added a comment. In 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

D21412: Add action to reset font size

2019-05-26 Thread Kishore Gopalakrishnan
kishoreg updated this revision to Diff 58678. kishoreg edited the test plan for this revision. kishoreg added a comment. Get the default font size correctly. REPOSITORY R39 KTextEditor CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21412?vs=58670&id=58678 REVISION DETAIL https:

D21412: Add action to reset font size

2019-05-25 Thread Kishore Gopalakrishnan
kishoreg created this revision. kishoreg added a reviewer: KTextEditor. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. kishoreg requested review of this revision. REVISION SUMMARY Add an action to reset the font size to the default and set