<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 19, 2016 at 7:10 PM, Marco Martin <span dir="ltr"><<a target="_blank" href="mailto:notmart@gmail.com">notmart@gmail.com</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span class="gmail-">On Mon, Sep 19, 2016 at 7:55 PM, David Edmundson<br>
<<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>> wrote:<br>
> <a target="_blank" rel="noreferrer" href="https://bugs.kde.org/show_bug.cgi?id=365967">https://bugs.kde.org/show_bug.<wbr>cgi?id=365967</a><br>
><br>
> It's a bug in that weather applet that it's rendering outside the area it<br>
> actually has - and that needs fixing anyway.<br>
> Introducing a new form factor won't solve that - in both cases you have to<br>
> do the exact same amount of work to that applet.<br>
<br>
</span>yes, just needs something for the applet to check (which at the moment<br>
doesn't have) i don't think it's fixable completely from systray side<br>
(most one could do, is clipping, which I would like to avoid)<br>
<span class="gmail-"><br>
>> IndicatorArea formfactor would have a Mutable immutability°, because<br>
>> the systray itself should always be able to both add and remove<br>
>> plasmoids in itself even when immutable (even Systemimmutable) it<br>
>> could still hide/disable the ui to do so when immutable, but must be<br>
>> programmatically able to do so.<br>
><br>
><br>
> 365618 and 365569 are basically the same thing right?<br>
<br>
</span>yes, same thing, entries not getting removed when locked.<br>
<span class="gmail-"><br>
> You're saying we need mutability to be different in system tray containment<br>
> compared to it's parent. Sounds fine. +1<br>
><br>
> But I don't see why we would tie that to the formFactor.<br>
> Semantically they're not remotely related, the only link is that the system<br>
> tray happens to need both.<br>
<br>
</span>perhaps you are right, semantically they are two things, not sure what<br>
could be used instead more "proper"(suggestions welcome), will think<br>
more about it.<br></blockquote><div><br></div><div>Long term, a virtual hook for immutability? <br></div><div></div><div></div><div></div><div><br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">
<br>
also, what do you suggest for 5.8? trying to temporarly unlock? <br></blockquote><br></div><div class="gmail_quote">We can try it.<br></div><div class="gmail_quote"><br></div><div class="gmail_quote"></div><div class="gmail_quote">This also works:<br><br>applet->pluginInfo().pluginName()) {<br>-            applet->destroy();<br>+            QMetaObject::invokeMethod(applet, "cleanUpAndDelete");<br><br></div><div class="gmail_quote">(in all 3 cases)<br></div><div class="gmail_quote"><div> <br></div><div>But it's equally messy, and has an issue that transient won't get set, so I don't like it.<br></div><div><br><br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">
--<br>
Marco Martin<br>
</blockquote></div><br></div></div>