Review Request: KolourPicker and PushButton

Aaron Seigo aseigo at kde.org
Fri Sep 12 17:08:54 CEST 2008


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


any work continuing on this, or should we just dump/ignore it?

- Aaron


On None, Loic Marteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.vidsolbach.de/r/61/
> -----------------------------------------------------------
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Here is an attempt to correctly deals with size constraints in kolourpicker.
> 
> The problem is with the KPushButtons inside the QProxyWidget's inherited Plasma::PushButton. They have a minimum size in their sizePolicy.
> So the key is here :
> m_grabButton->nativeWidget()->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);
> 
> Perhaps we have to put this line of code directly in Plasma::PushButton to avoid problems with other applets ?
> 
> I have put some code too to define the size constraints policy in panel mode.
> 
> 
> Cheers
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeplasmoids/applets/kolourpicker/kolourpicker.h
>   /trunk/KDE/kdeplasmoids/applets/kolourpicker/kolourpicker.cpp
> 
> Diff: http://reviewboard.vidsolbach.de/r/61/diff
> 
> 
> Testing
> -------
> 
> Works for me
> 
> 
> Thanks,
> 
> Loic
> 
>



More information about the Plasma-devel mailing list