Summary: | Convention of passing "org.kde.plasma:force-create" as argument to Containment::createApplet is crap | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Eike Hein <hein> |
Component: | general | Assignee: | David Edmundson <kde> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | jr, kde, kde, nate, notmart, plasma-bugs |
Priority: | NOR | ||
Version: | master | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Eike Hein
2018-05-09 08:51:26 UTC
I should definitely remove the arg once it has done what it wanted to do. On creation it checks the arguments and then sets a dynamic property onto itself. After that it could remove the argument. I think I'll go adding a property on systray/grouping containments like "dynamiccontent" which would replace the argument. However... I think this would be resolved "properly" once we have sandboxed stuff, those two nesting containments would be replaced by out of process shells which would never be locked and bam, done (In reply to Marco Martin from comment #3) > I think I'll go adding a property on systray/grouping containments like > "dynamiccontent" which would replace the argument. hmm, easy for Containment::addApplet, but not for Corona::createContainment, which would need to access the containment metadata before creating it, which is possible but very dirty Could you explain a bit more context? What is this org.kde.plasma:force-create stuff even used for? System Tray, so it can create applets even when the shell itself is immutable. I think the Grouping Plasmoid also uses it for the same purpose. |