Review Request: Deactivate Dashboard by Click on Background

Aaron Seigo aseigo at kde.org
Thu Mar 6 18:25:00 CET 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://mattr.info/r/260/#review251
-----------------------------------------------------------

Ship it!


the idea is fine, imho.


trunk/KDE/kdebase/workspace/plasma/plasma/dashboardview.cpp
<http://mattr.info/r/260/#comment188>

    should probably check for a non-right click so we can provide a context menu as well.
    
    also, why an event filter vs reimplementing one of the mouse *Event methods?


- Aaron


On 2008-03-06 08:36:20, Stephan Binner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://mattr.info/r/260/
> -----------------------------------------------------------
> 
> (Updated 2008-03-06 08:36:20)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Deactivate dashboard view mode on click on background (like Mac OS).
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/plasma/plasma/dashboardview.cpp
> 
> Diff: http://mattr.info/r/260/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Stephan
> 
>



More information about the Panel-devel mailing list