Review Request 127241: Change default CTRL+Click jump to definition instead of declaration

Pedro Ferreira esteaqui at hotmail.com
Tue Mar 1 15:49:09 UTC 2016


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

(Updated March 1, 2016, 3:49 p.m.)


Review request for KDevelop.


Changes
-------

Corrected some issues Milian pointed out.


Repository: kdevplatform


Description
-------

The current default behaviour on CTRL+Click causes the editor do jump to the function's declaration. This patch changes that to jump to definition instead.
I initially wrote this as a personal hack, but from IRC conversations, it seems that I'm not the only one who's annoyed at this behaviour, so here's my upstream attempt.
Do note that I know little of the sorrounding code. If you believe further changes are required, I'll need some guidance on that.


Diffs (updated)
-----

  plugins/contextbrowser/browsemanager.h f631189 
  plugins/contextbrowser/browsemanager.cpp 1c5cc70 

Diff: https://git.reviewboard.kde.org/r/127241/diff/


Testing
-------

It works on my projects and on my KDevPlatform project.


Thanks,

Pedro Ferreira

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


More information about the KDevelop-devel mailing list