Review Request 127669: Add back Source Browse Mode as a menu action

[Igor Kushnir]

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

(Updated May 19, 2016, 4:28 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDevelop.


Changes
-------

Submitted with commit 4cc756784e03640562665b1163552b7199004960 by Gleb Popov on behalf of Igor Kushnir to branch 5.0.


Repository: kdevplatform


Description
-------

A recent commit "Remove confusing button in contextbrowser" removed this mode.
However this mode is useful for quick navigation when reading/debugging code.
This partially reverts commit aeb3a26837fe1510bbb5f49fffaad8d560498f65.

A menu item is not nearly as distracting and confusing as a fickle tool button.
So this commit does not reintroduce BUG 358498.


Diffs
-----

  plugins/contextbrowser/browsemanager.h efe39cdf077316602a70ae4133ea3edc499dbb38 
  plugins/contextbrowser/browsemanager.cpp 0d8f6cb494c268a49ec46fd6d3ef050598d18250 
  plugins/contextbrowser/contextbrowser.cpp f8ab2aa9f46be9482475d913d506531b353822bf 
  plugins/contextbrowser/kdevcontextbrowser.rc d6ddeada755ab9f08b9948498ec727a6535af1b5 

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


Testing
-------

Built, installed.
Also built and installed kdevelop with changes from https://git.reviewboard.kde.org/r/127670/
Verified the presence of the action in the Navigation menu.
Verified that a shortcut can be assigned and works.
Verified that the action works as expected: enables/disables Source Browse Mode.


Thanks,

Igor Kushnir

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