[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Plasma/PopupApplet and Plasma::PackageStructure
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2010-02-27 17:29:22
Message-ID: 201002270929.22711.aseigo () kde ! org
[Download RAW message or body]

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
[prev in list] [next in list] [prev in thread] [next in thread]