Review Request: Prevent code navigation tooltip hiding when moving mouse over the identifier.

Maciej Cencora m.cencora at gmail.com
Thu Jul 12 07:47:13 UTC 2012



> On July 11, 2012, 10:26 p.m., Aleix Pol Gonzalez wrote:
> > How can we reproduce the problem this patch is fixing?

Preconditions:
Code browser widget is hidden.

Steps to reproduce:
1) In editor move mouse pointer over an identifier
2) Wait for code navigation tooltip to show up
3) Move mouse pointer around but still over this identifier, in direction away from tooltip 

Result:
Tooltip disappears for a moment (<1s), and shows up again

Expected behavior:
Tooltip does not disappear while moving mouse over the identifier


- Maciej


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


On July 1, 2012, 5:27 p.m., Maciej Cencora wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105403/
> -----------------------------------------------------------
> 
> (Updated July 1, 2012, 5:27 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> This works by looking up the item's Range in DUChain and adding its geometry boundaries as extra rectangle to tooltop.
> 
> If there's another way to get line height (in pixels) in kate I'd be more than happy to use it instead of the hack I wrote.
> 
> 
> Diffs
> -----
> 
>   language/duchain/duchainutils.h d3cc1af 
>   language/duchain/duchainutils.cpp 1879540 
>   plugins/contextbrowser/contextbrowser.cpp 4056ac0 
> 
> Diff: http://git.reviewboard.kde.org/r/105403/diff/
> 
> 
> Testing
> -------
> 
> Manual testing performed
> 
> 
> Thanks,
> 
> Maciej Cencora
> 
>

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


More information about the KDevelop-devel mailing list