Review Request: Moving of plasmoids on panel

[Marco Martin]

> On 2008-03-07 02:18:06, Will Stephenson wrote:
> > It's ok, but you can only change the relative position of applets with it.  If you have an empty panel there are no other applets to exchange position with, so you can't move them to the other end of the panel (tested with horizontal and vertical panels).
> > 
> > A rather random observation: An implementation that would work with non-panel containments could delegate the relayout to the layout, by passing the hoverevent and the applet sizes to the layout, and saying "you figure out where this should go and where the other contents have to move to".

freely moving the aplpets.. is really desired? it would require to not use linear layouts anymore, i would stick with rearranging concept.
for moving an applet at the end of the panel it could be some automation to the "spacer" applet, like automatically creating one when an applet is dragged far from its natural position. or simply making the spacer applet available in the list like is now, but the problem is that it should not be possible to add it in the desktop or in containments that have a free layout because it relly doesn't make sense here.


- Marco


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


On 2008-03-06 16:13:31, Stephan Binner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://mattr.info/r/261/
> -----------------------------------------------------------
> 
> (Updated 2008-03-06 16:13:31)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Simple implementation to allow rearranging of plasmoids on panels (#154119).
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/workspace/libs/plasma/containment.cpp
>   trunk/KDE/kdebase/workspace/libs/plasma/containment.h
> 
> Diff: http://mattr.info/r/261/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Stephan
> 
>