[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: KColorPicker and other plasmoids
From:       "=?ISO-8859-1?Q?Lo=EFc_Marteau?=" <loic.marteau () gmail ! com>
Date:       2008-06-19 19:37:49
Message-ID: 9c369230806191237n10cf85d8gc8cf5e82d6dbdd46 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

[Attachment #4 (text/plain)]

Hey !

I've posted a patch on the review board to deal with this
http://reviewboard.vidsolbach.de/r/61/

the key is to redefine sizePolicy for the KPushButton inside the
Plasma:PushButton to don't let him have a minimum size.

Is it ok to commit ?

Cheers

On Wed, Jun 18, 2008 at 5:16 PM, Riccardo Iaconelli <riccardo@kde.org>
wrote:

> Il Wednesday 18 June 2008 16:06:36 Anne-Marie Mahfouf ha scritto:
> > On Wednesday 18 June 2008 03:38:03 pm Toussis Manolis wrote:
> > > I already send him an email. thanks for your time.
> >
> > I am investigating this further by doing some test cases and will report
> to
> > the list.
> >
> > So far it seems that 2 Plasma::PushButton don't layout nicely while 2
> > Plasma::Label do.
> > Another thing about the Kolour Picker is that the circle on the bottom
> > button (the circle has the latest picked color)  seems to have
> disappeared
> > when the change QPushButton ->K was done.
> >
> > So if anyone wants to help, you can start confirming this and then
> finding
> > out why PushButtons do not layout.
>
> For my applet, I had to run the following code each time I added or removed
> an
> applet from the layout, to make the widgets not go outside of the applet,
> or
> anyways to get a correct layouting behiavour:
>
>    updateGeometry();
>    resize(sizeHint(Qt::PreferredSize));
>    setMinimumSize(sizeHint(Qt::MinimumSize));
>
> I put this in a function. The fact that we have to do that is pretty bad
> though. :-(
>
> Bye,
> -Riccardo
> --
> GPG key:
> 3D0F6376
> When encrypting, please encrypt also for this subkey:
> 9EBD7FE1
> -----
> Pace Peace Paix Paz Frieden Pax Pokój Friður Fred Béke 和平
> Hasiti Lapé Hetep Malu Mир Wolakota Santiphap Irini Peoch שלום
> Shanti Vrede Baris Rój Mír Taika Rongo Sulh Mir Py'guapy 평화
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
>

[Attachment #5 (text/html)]

Hey !<br><br>I&#39;ve posted a patch on the review board to deal with this<br><a \
href="http://reviewboard.vidsolbach.de/r/61/">http://reviewboard.vidsolbach.de/r/61/</a><br><br>the \
key is to redefine sizePolicy for the KPushButton inside the Plasma:PushButton to \
don&#39;t let him have a minimum size.<br> <br>Is it ok to commit \
?<br><br>Cheers<br><br><div class="gmail_quote">On Wed, Jun 18, 2008 at 5:16 PM, \
Riccardo Iaconelli &lt;<a href="mailto:riccardo@kde.org">riccardo@kde.org</a>&gt; \
wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, \
204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Il Wednesday 18 June 2008 \
16:06:36 Anne-Marie Mahfouf ha scritto:<br> <div class="Ih2E3d">&gt; On Wednesday 18 \
June 2008 03:38:03 pm Toussis Manolis wrote:<br> &gt; &gt; I already send him an \
email. thanks for your time.<br> &gt;<br>
&gt; I am investigating this further by doing some test cases and will report to<br>
&gt; the list.<br>
&gt;<br>
&gt; So far it seems that 2 Plasma::PushButton don&#39;t layout nicely while 2<br>
&gt; Plasma::Label do.<br>
&gt; Another thing about the Kolour Picker is that the circle on the bottom<br>
&gt; button (the circle has the latest picked color) &nbsp;seems to have \
disappeared<br> &gt; when the change QPushButton -&gt;K was done.<br>
&gt;<br>
&gt; So if anyone wants to help, you can start confirming this and then finding<br>
&gt; out why PushButtons do not layout.<br>
<br>
</div>For my applet, I had to run the following code each time I added or removed \
an<br> applet from the layout, to make the widgets not go outside of the applet, \
or<br> anyways to get a correct layouting behiavour:<br>
<br>
 &nbsp; &nbsp;updateGeometry();<br>
 &nbsp; &nbsp;resize(sizeHint(Qt::PreferredSize));<br>
 &nbsp; &nbsp;setMinimumSize(sizeHint(Qt::MinimumSize));<br>
<br>
I put this in a function. The fact that we have to do that is pretty bad<br>
though. :-(<br>
<div class="Ih2E3d"><br>
Bye,<br>
-Riccardo<br>
--<br>
GPG key:<br>
3D0F6376<br>
When encrypting, please encrypt also for this subkey:<br>
9EBD7FE1<br>
-----<br>
Pace Peace Paix Paz Frieden Pax Pokój Friður Fred Béke 和平<br>
Hasiti Lapé Hetep Malu Mир Wolakota Santiphap Irini Peoch שלום<br>
Shanti Vrede Baris Rój Mír Taika Rongo Sulh Mir Py&#39;guapy 평화<br>
_______________________________________________<br>
</div><div><div></div><div class="Wj3C7c">Panel-devel mailing list<br>
<a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/panel-devel" \
target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br> \
</div></div></blockquote></div><br>



_______________________________________________
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