<br><div><span class="gmail_quote">On 6/2/08, <b class="gmail_sendername">Marco Martin</b> <<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>> wrote:</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>plasma::square is not really good, because when using a vertical panel it<br> becomes really big and the square things will become really huge and you<br> would end up with like the kmenu that takes a quarter of the panel height.<br>
i think it would be needed something more like what is in the kickoff applet<br> now, even if a bit less intuitive. when it's in an horizontal panel the width<br> can't be more than the height (but height can be more than width)<br>
and the opposite in a vertical panel<br> <br> Cheers,<br> <br>Marco Martin</blockquote><div><br><br>Yes that's what I said a bit further. This result to have a more complicated code if we think that this problem with big panel isn't acceptable (but who really want big panel ?).<br>
<br>So what is the solution ? (I don't have personal opinion for this).<br>- keep this code and don't care of big panel problem<br>- Adapt plasma:square related code in the core of plasma and keep this code in icon based plasmoid<br>
- add a <span class="q">special constructor or method at Plasma:Icon as proposed by Aaron (can this solution be adapted to deal with big panel issue ?)<br>- use a generic virtual class<br><br>Cheers,<br><br>Loic<br></span> </div>
<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> ><br> > Perhaps the problem with this is that we dont put max size constraint<br>
> when panel is big. I dont know if it is important. If it is, the<br> > solution have to be directly in plasma::square related code or can your<br> > proposed solution deal with this ?<br> ><br> <br> > Loic<br>
<br> <br> <br>_______________________________________________<br> 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">https://mail.kde.org/mailman/listinfo/panel-devel</a><br>
</blockquote></div><br>