Review Request: Added feature in "showdesktop" widget to minimize all opened window when any file/folder is dragged from the opened window and hover over "showdesktop" widget icon.

[Shantanu Tushar Jha]

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

Ship it!


Good idea, just address the issues Aaron and Marco pointed out. Also, Plasma follows the kdelibs coding guidelines, you may want to go through http://techbase.kde.org/Policies/Kdelibs_Coding_Style . 

- Shantanu


On 2010-08-17 19:01:14, Sinny Kumari wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5060/
> -----------------------------------------------------------
> 
> (Updated 2010-08-17 19:01:14)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Earlier, when I was trying to drag any file/folder on desktop from an opened window , i was unable to do it because there was no option to minimize all opened window when an item is dragged.
> So, I added  feature to "minimize all opened window" in "showdesktop" widget when any item is dragged from opened window.
> It's possible by hovering the mouse in dragged condition over "showdesktop" widget icon in panel.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdeplasma-addons/applets/showdesktop/showdesktop.h 1162122 
>   trunk/KDE/kdeplasma-addons/applets/showdesktop/showdesktop.cpp 1162122 
> 
> Diff: http://reviewboard.kde.org/r/5060/diff
> 
> 
> Testing
> -------
> 
> It works fine with trunk!
> 
> 
> Thanks,
> 
> Sinny
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100818/d2ae6b02/attachment.htm