Review Request 113264: Patch for Bug 273385 : Provide a shortcut for switching focus between split views

Meenakshi Khorana akshvi at gmail.com
Thu Oct 17 07:01:39 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113264/
-----------------------------------------------------------

(Updated Oct. 17, 2013, 7:01 a.m.)


Review request for KDevelop.


Changes
-------

Fixed the issues opened by Kfunk.

- Added Spaces between '='

- Appended "View" everywhere. Eg In "&Next Split View"


Repository: kdevplatform


Description
-------

Patch for Bug 273385 : Provide a shortcut for switching focus between split views

Provided a shortcut (key combination as well as on menu option) for switching between split views.Implemented the same by providing shortcut (Ctrl + Shift + P for moving to previous split and Ctrl + Shift + N for moving to next split) along with menu options in Window drop-down menu


Diffs (updated)
-----

  shell/mainwindow_actions.cpp 8ef2278 
  shell/mainwindow_p.h 2cd0102 
  shell/mainwindow_p.cpp 82d7897 

Diff: http://git.reviewboard.kde.org/r/113264/diff/


Testing
-------

Compiled and tested successfully on my local machine


File Attachments (updated)
----------------

Screenshot for new options added in Window menu
  http://git.reviewboard.kde.org/media/uploaded/files/2013/10/16/5eac53d7-c6e1-4b40-bda7-6087381f51d1__bug_273385.jpg
Sreenshot for revised menu options to switch between splits
  http://git.reviewboard.kde.org/media/uploaded/files/2013/10/17/f26d7b9b-de60-4246-9e9c-16719958540c__switch_bw_split.jpg
Screenshot for revised menu option to switch between splits
  http://git.reviewboard.kde.org/media/uploaded/files/2013/10/17/26e6d869-c69f-45d9-a840-188a239ffe2f__switch_bw_split.jpg


Thanks,

Meenakshi Khorana

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20131017/b31acac2/attachment.html>


More information about the KDevelop-devel mailing list