On February 27, 2010, Yuen Hoe Lim wrote: > Is this a bug or is this intentional? If the latter, then is it acceptable > to simply do
looking at your last commit to plasmate, i see why you asked this. all plasmoids should have Plasma/Applet in the ServiceTypes= entry of the .desktop file, since all Plasma/PopupApplets are also Plasma/Applets. it means we have to constantly search for Plasma/PopupApplets to build a complete list of applets, and PopupApplet is just a hint for scripting purposes anyways. right now Plasma::Applet does search for Plasma/PopupApplet as well as Plasma/Applet, so this is worked around pretty thoroughly. but i do wonder if that code should be made to only rely on Plasma/Applet. the only place Plasma/PopupApplet should be needed is in Applet::loadPlasmoid. the danger is that scripted plasmoids that used to work might stop being shown in the applet listings if they just had ServiceTypes=Plasma/PopupApplet. however, the documentation on techbase that mentions this as a possibility does also state that it must include Plasma/Applet. so i'm considering removing the references to Plasma/PopupApplet in Plasma::Applet except for where it is used to actually create the correct base class. Richard: what do you think? -- Aaron J. Seigo humru othro a kohnu se GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 KDE core developer sponsored by Qt Development Frameworks _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel