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

List:       kde-panel-devel
Subject:    Re: Review Request: Comic applet proted to PopupApplet
From:       Giulio Camuffo <giuliocamuffo () gmail ! com>
Date:       2009-10-31 9:53:40
Message-ID: 9f3bf61b0910310253m5fcb2d56y6afdcd5c363dde80 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


2009/10/31 Aaron J. Seigo <aseigo@kde.org>

> On October 30, 2009, Giulio Camuffo wrote:
> > In data venerd=EC 30 ottobre 2009 18:03:07, Aaron J. Seigo ha scritto:
> > : > On October 30, 2009, Giulio Camuffo wrote:
> > : >
> > > > So this patch fixes this.
> > >
> > > i think you forgot the patch
>
> it looks good except for setting a minimum size of 20x20. that's a bit to=
o
> big
> for people who will be wanting 16px tall panels :) set it to 8x8 if
> anything?
>
> you're right. maybe it's better not to set it entirely?


Giulio

[Attachment #5 (text/html)]

<br><br><div class="gmail_quote">2009/10/31 Aaron J. Seigo <span dir="ltr">&lt;<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>&gt;</span><br><blockquote \
class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt \
0pt 0.8ex; padding-left: 1ex;"> <div class="im">On October 30, 2009, Giulio Camuffo \
wrote:<br> &gt; In data venerd́ 30 ottobre 2009 18:03:07, Aaron J. Seigo ha \
scritto:<br> &gt; : &gt; On October 30, 2009, Giulio Camuffo wrote:<br>
&gt; : &gt;<br>
&gt; &gt; &gt; So this patch fixes this.<br>
&gt; &gt;<br>
&gt; &gt; i think you forgot the patch<br>
<br>
</div>it looks good except for setting a minimum size of 20x20. that&#39;s a bit too \
big<br> for people who will be wanting 16px tall panels :) set it to 8x8 if \
anything?<br> <div class="im"><br></div></blockquote><div>you&#39;re right. maybe \
it&#39;s better not to set it entirely?<br> <br><br>Giulio</div></div><br>



_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


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

Configure | About | News | Add a list | Sponsored by KoreLogic