[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request: drag to desktop/panel support in kickoff
From: Marco Martin <notmart () gmail ! com>
Date: 2008-02-26 15:15:20
Message-ID: 20080226151520.32683.50775 () localhost
[Download RAW message or body]
> On 2008-02-25 16:09:43, Aaron Seigo wrote:
> > minor formatting issues, otherwise go for it.
well, i really suck for these things :D
as for the search problem i had forgotten a setDragEnabled, whooops :P
- Marco
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://mattr.info/r/220/#review197
-----------------------------------------------------------
On 2008-02-25 15:45:35, Marco Martin wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://mattr.info/r/220/
> -----------------------------------------------------------
>
> (Updated 2008-02-25 15:45:35)
>
>
> Review request for Plasma.
>
>
> Summary
> -------
>
> This add the possibility to drag items from kickoff to dektop or to panel, by \
> making the models descend from a KickoffModel class that adds the support to drag \
> and drop. It works with items in all tabs (except for applications categories and \
> the leave tab, wich doesn't make sense) It has the problem that it has three carbon \
> copy classes KickoffModel, KickoffProxyModel and KickoffAbstractModel, that kinda \
> sucks :( is there a way to do it a little bit better? in third revision cleaned up \
> the code a little bit (oh, joys of copy and paste, oh well..)
>
> Diffs
> -----
>
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffproxymodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffproxymodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffabstractmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/favoritesmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/kickoffabstractmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/favoritesmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/applicationmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/CMakeLists.txt
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/applicationmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/recentlyusedmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/recentlyusedmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/searchmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/searchmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/systemmodel.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/systemmodel.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/simpleapplet/simpleapplet.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/flipscrollview.h
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/flipscrollview.cpp
> /trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/urlitemview.cpp
>
> Diff: http://mattr.info/r/220/diff
>
>
> Testing
> -------
>
> tried to drag on desktop and panel various items from various tabs
> the only problem is in the search result, when i try to drag an item nothing \
> happens and "Can't select indexes from different model or with different parents"
> is written on standard output
>
>
> Thanks,
>
> Marco
>
>
_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic