Review Request 119214: Respect the "DoNotCloseOnCursorMove" property of navigation widgets

Denis Steckelmacher steckdenis at
Thu Jul 10 15:18:54 UTC 2014

This is an automatically generated e-mail. To reply, visit:

Review request for KDevelop.

Repository: kdevplatform


When the "DoNotCloseOnCursorMove" property of a navigation widget is set, the widget should not be closed when the text cursor moves (if the current document is modified by the widget, for instance).

This patch fixes a small error in contextbrowser.cpp: the cursorPositionChanged signal was connected to hideToolTip only if the "DoNotCloseOnCursorMove" was not set, but this signal was never disconnected. This made "DoNotCloseOnCursorMove" work as long as the user does not open any widget on which the property is not set (such a widget connected the signal with no chance of having it disconnected later).

(note: do I commit this patch in master or in frameworks?)


  plugins/contextbrowser/contextbrowser.cpp cf03b16 



The helper widgets of QML/JS (for instance, the one that allows to select a color) now work reliably and are not closed when they update the document, even after an extensive editing session during which many navigation widgets have been opened.


Denis Steckelmacher

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the KDevelop-devel mailing list