Review Request: Debug tooltip overlap

[Julien Desgats]

> On March 12, 2011, 2:01 p.m., Milian Wolff wrote:
> > looks good but what bug are you talking about? you did not include any link

I did put the link for the bug, in section "Bugs". Anyway, it's http://bugs.kde.org/show_bug.cgi?id=268196
And for the discussion, it is archived at https://barney.cs.uni-potsdam.de/mailman/private/kdevelop-devel/2010-November/039241.html


- Julien


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


On March 11, 2011, 1:09 p.m., Julien Desgats wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100840/
> -----------------------------------------------------------
> 
> (Updated March 11, 2011, 1:09 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> This little patch fixes the overlapping problem with debug tooltips. According to what we said in ML some months ago (topic "Debugger and duchain hints overlaps") debug tooltips are shown only on debug area. 
> 
> However, using ActiveToolTip::showToolTip instead of just show() avoids overlap, so we could also show both tooltips, replace "ActiveToolTip::showToolTip(this, 0.0);" by "ActiveToolTip::showToolTip(this, 1.0);" to see the result. The debug tooltip will be displayed before navigation tooltip, which will be unreachable (it will disappear as soon as the cursor move).
> 
> When working on that, I discover that ActiveToolTip::showToolTip does not behave like the documentation says (for exclusive mode), see bug link. In order to work correctly, this patch need bug patch too.
> 
> 
> This addresses bug 268196.
>     http://bugs.kde.org/show_bug.cgi?id=268196
> 
> 
> Diffs
> -----
> 
>   debugger/variable/variablecollection.cpp 5edd4a9 
>   debugger/variable/variabletooltip.cpp 58ce558 
>   interfaces/iuicontroller.h 49d5a10 
> 
> Diff: http://git.reviewboard.kde.org/r/100840/diff
> 
> 
> Testing
> -------
> 
> Test debugging with some projects.
> 
> 
> Thanks,
> 
> Julien
> 
>

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