Review Request: Drag taskbar entries into the pager

Aaron Seigo aseigo at kde.org
Wed Dec 17 01:18:18 CET 2008


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/312/#review302
-----------------------------------------------------------


ok, there is now:

static QString Task::mimetype()
static WId Task::idFromMimeData(QMimeData *mimeData, bool *ok = 0)
void Task::addMimedData(QMimeData *mimeData)

note that the last one is not static =)

this should significantly ease the impact of this patch

- Aaron


On 2008-12-16 14:38:29, Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/312/
> -----------------------------------------------------------
> 
> (Updated 2008-12-16 14:38:29)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Adds to KDE 4 the missing feature of being able to drag taskbar entries into the pager to move them between desktops
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.h
>   /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.h
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/abstracttaskitem.cpp
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.h
>   /trunk/KDE/kdebase/workspace/plasma/applets/tasks/windowtaskitem.cpp
> 
> Diff: http://reviewboard.vidsolbach.de/r/312/diff
> 
> 
> Testing
> -------
> 
> Played with it for a while and did not explode
> 
> 
> Thanks,
> 
> Albert
> 
>



More information about the Plasma-devel mailing list