Review Request 125908: taskmanager: Make the windows in tooltip scrollable

[Eike Hein]

> On Nov. 25, 2015, 1:04 p.m., Eike Hein wrote:
> > Ship It!
> 
> David Rosca wrote:
>     Hmm, it no longer works because the tooltip size is not bound by the available screen size.
>     So it needs fix in plasma-framework first.
> 
> Marco Martin wrote:
>     hmm, can you explain what the problem in tooltip management is?

I tested the patch before I gave the Ship It!, and it worked just fine for me with latest plasma-framework. The Tooltip dialog docked at the screen edges left and right and the contents scrolled.


- Eike


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


On Nov. 5, 2015, 9:05 a.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125908/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2015, 9:05 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Make the windows in tooltip scrollable when it doesn't fit on the screen.
> 
> 
> Diffs
> -----
> 
>   applets/taskmanager/package/contents/ui/ToolTipDelegate.qml f7084c7 
> 
> Diff: https://git.reviewboard.kde.org/r/125908/diff/
> 
> 
> Testing
> -------
> 
> Works fine. One thing that would be great is to allow vertical wheel scrolling, but for that KQuickControlsAddons.EventGenerator needs to support wheel events (custom wheel handling would be too complex for Flickable)
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151125/eb051bf7/attachment-0001.html>