<br><br><div class="gmail_quote">On Tue, Dec 2, 2008 at 12:02 PM, Christian Mollekopf <span dir="ltr">&lt;<a href="mailto:chrigi_1@fastmail.fm">chrigi_1@fastmail.fm</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Am 02.12.2008, 11:43 Uhr, schrieb Marco Martin &lt;<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>&gt;:<br>
<div class="Ih2E3d"><br>
&gt; On Tuesday 02 December 2008, Alexis Ménard wrote:<br>
&gt;&gt; And the layout is recreated again and again, i am sure we can find a<br>
&gt;&gt; solution for that.<br>
&gt;<br>
&gt; time ago i tried to remove that thing, it resulted in empty holes still<br>
&gt; sized<br>
&gt; as there were items in it, so then tried with a simple qt program and the<br>
&gt; problem wasn&#39;t there, so it must be a problem somewhere in<br>
&gt; layoutwidget.cpp,<br>
&gt; but it&#39;s pretty complex, really wasn&#39;t able to track it down<br>
<br>
</div>Since i did most of this stuff i should attend the meeting as well =)<br>
Good for me is between 20.00- 24.00 GMT+1, today for instance would be<br>
good.<br>
<br>
The layout recreation is a workaround since the QGraphicsGridLayout<br>
doesn&#39;t remove the items properly.<br>
I think Aaron and Marco agreed on this workaround an we will have to keep<br>
it that way until it&#39;s fixed in QT.<br>
</blockquote><div><br>If the problem is in Qt then i will fix it, it&#39;s my job. And removing items should work and must work...<br>I have already used QGraphicsGridLayout and it should work as far as i know and the last time i used it.<br>
Anyway if the bug exist why don&#39;t you write to the support of Qt Software about that?<br>I have actually screen all QGraphicsLayout tasks and there are no such of things. So the bug will never be fixed if it exist.<br>
<br>Anyway i will take a look at it...<br><br>&nbsp;</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
Regards,<br>
<br>
Christian<br>
<div><div></div><div class="Wj3C7c"><br>
&gt;<br>
&gt;&gt; 2008/12/2 Aaron J. Seigo &lt;<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>&gt;<br>
&gt;&gt;<br>
&gt;&gt; &gt; hi...<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; i&#39;d like to have a tasks widget hacking day.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; the reason for this i that the code is something of a mess internally<br>
&gt;&gt; and<br>
&gt;&gt; &gt; imho<br>
&gt;&gt; &gt; it&#39;s unmaintainable in its current state. i say this because changing<br>
&gt;&gt; &gt; little<br>
&gt;&gt; &gt; things in one place often create rather unexpected results; there&#39;s a<br>
&gt;&gt; &gt; good amount of unreachable code; there&#39;s what looks to be some pretty<br>
&gt;&gt; &gt; obvious memory leaks (e.g. Tasks::m_groupTaskItems never seems to have<br>
&gt;&gt; &gt; items removed<br>
&gt;&gt; &gt; from it!); Tasks does bookkeeping, but for only some things, and this<br>
&gt;&gt; &gt; bookkeeping is controlled from Tasks, LayoutWidget *and*<br>
&gt;&gt; TaskGroupItem!<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; there are bugs that sometimes results in &quot;holes&quot; showing up in the<br>
&gt;&gt; &gt; layout, grouping not working reliably on start up and more.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; if we ship with it like this, we will hate ourselves later.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; unfortunately, i&#39;m not particularly sure what all the reasons and<br>
&gt;&gt; &gt; rationals are for some of the code decisions.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; so if you have been working on the tasks widget in the last month or<br>
&gt;&gt; two,<br>
&gt;&gt; &gt; please respond in this thread with when would be a good time and day<br>
&gt;&gt; for<br>
&gt;&gt; &gt; you<br>
&gt;&gt; &gt; so we can get together on irc and sort this thing out. =)<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; --<br>
&gt;&gt; &gt; Aaron J. Seigo<br>
&gt;&gt; &gt; humru othro a kohnu se<br>
&gt;&gt; &gt; GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA &nbsp;EE75 D6B7 2EB1 A7F1 DB43<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; KDE core developer sponsored by Qt Software<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; _______________________________________________<br>
&gt;&gt; &gt; Plasma-devel mailing list<br>
&gt;&gt; &gt; <a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
&gt;&gt; &gt; <a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
&gt;<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; Plasma-devel mailing list<br>
&gt; <a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
&gt; <a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
<br>
<br>
<br>
</div></div><font color="#888888">--<br>
Erstellt mit Operas revolutionärem E-Mail-Modul: <a href="http://www.opera.com/mail/" target="_blank">http://www.opera.com/mail/</a><br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<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" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br>