[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"><<a \
href="mailto:aseigo@kde.org">aseigo@kde.org</a>></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> > In data venerd́ 30 ottobre 2009 18:03:07, Aaron J. Seigo ha \
scritto:<br> > : > On October 30, 2009, Giulio Camuffo wrote:<br>
> : ><br>
> > > So this patch fixes this.<br>
> ><br>
> > i think you forgot the patch<br>
<br>
</div>it looks good except for setting a minimum size of 20x20. that'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're right. maybe \
it'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