Aaron J. Seigo дµÀ:<br>> On Thursday 23 July 2009, ÅËÎÀƽ(Peter Pan) wrote:<br>>> Taht will cover the icon and looks mussy.<br>>> I prefer to let it over the text.<br>> <br>> hm.. ok, how about on the right side of the button then, where the # of <br>
> windows usually appears when there are several windows grouped?<br>> <br>> also as for this:<br>> <br>> + QRectF rect = parentGroup()->tasksLayout()->contentsRect();<br>> + int rowCount = parentGroup()->tasksLayout()->rowCount();<br>
> + qreal height = rect.height() / rowCount; <br>> + QSizeF size(height, height);<br>> <br>> it should be able to just position it relative to the task button itself, <br>> something like this which would make it consistent with the group number <br>
> drawing as well:<br>> <br>> Plasma::FrameSvg *itemBackground = m_applet->itemBackground();<br>> QSizeF busySize(itemBackground->elementSize(expanderElement()));<br>> QRectF animationRect(QPointF(size().width() - busySize.width(), (size.height() <br>
> - busySize.height()) / 2), busySize);<br>> <br>> even with this code, however, if the size of the button changes while the app <br>> is launching the busy widget will no longer be in the right place.<br>> <br>
> so ... either the positioning code should be put into a separate method and be <br>> called from both startStartupAnimation and resizeEvent or a layout should be <br>> used to manage that for us. a layout is probably overkill in this situation, <br>
> though.<br>> <br>> <br>> <br>> ------------------------------------------------------------------------<br>> <br>> _______________________________________________<br>> Plasma-devel mailing list<br>
> <a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>> <a href="https://mail.kde.org/mailman/listinfo/plasma-devel">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br><br>Fisrt, I prefer to use Plasma::BusyWidget, consistent with others.<br>
<br>Second, I prefer to replace icon when the app is starting up, like<br>firefox when it is opening a new website.<br><br>Third, I set the BusyWidgest's geometry in AbstractTaskItem::drawTask(),<br>so it can get the proper geometry.<br>
<br>Best regards!<br><br><br>-- <br>ÅËÎÀƽ(Peter Pan)<br>