Review Request 108908: Add menu action and shortcut for switch to buddy plugin

[Milian Wolff]

> On Feb. 24, 2013, 2:46 p.m., David Nolden wrote:
> > The C++ plugin already has a "Switch Header/Source" shortcut. Where's the difference?
> > 
> > Also I think exposing a string like "Buddy Document" to the user is confusing, something like "Related Document" might work better.
> 
> André Stein wrote:
>     I wasn't aware of that shortcut and can't find it.. Or do you mean the switch to xx.cpp/xx.h context menu extension? That is provided by the Switch to Buddy plugin and this request adds the appropriate shortcut. 
>     
>     About the name: If "related document" is found to be better the whole plugin should be renamed then.

David: We only have switch declaration/definition which is not always what one wants. Esp. if one is working on pimpled classes, there is currently no shortcut to switch to the .h file if your cursor is in the context of the private class.


- Milian


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


On March 8, 2013, 10:50 a.m., André Stein wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108908/
> -----------------------------------------------------------
> 
> (Updated March 8, 2013, 10:50 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> As Milian suggested some time ago I implemented a menu action (in the Navigation menu) which allows swichting to the buddy document. A global action has been added too which allows to specify a shortcut for that action (Ctrl+Shift+s per default).
> 
> As I don't have much time right now I didn't implement Milian's idea to display a document switcher window when multiple buddies are available. Right now the plugin just switches to the first, "primary" buddy document.
> 
> 
> Diffs
> -----
> 
>   plugins/switchtobuddy/switchtobuddyplugin.cpp 2a3073a 
>   plugins/switchtobuddy/switchtobuddyplugin.h efa7f1e 
>   plugins/switchtobuddy/kdevswitchtobuddy.rc PRE-CREATION 
>   interfaces/ibuddydocumentfinder.h 05fb7ee 
>   plugins/switchtobuddy/CMakeLists.txt 795b5ed 
> 
> Diff: http://git.reviewboard.kde.org/r/108908/diff/
> 
> 
> Testing
> -------
> 
> Manual testing.
> 
> 
> Thanks,
> 
> André Stein
> 
>

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