Projects/Plasma/Tokamak1: Difference between revisions

    From KDE TechBase
    (constraintUpdated()->constraintEvent()+protected)
     
    (50 intermediate revisions by 6 users not shown)
    Line 14: Line 14:
    * <s>slots -> Q_PRIVATE_SLOT</s>
    * <s>slots -> Q_PRIVATE_SLOT</s>


    === Plasma::Corona (Marco Martin) ===
    === <s>Plasma::Corona</s> ===


    * <s>Remove constructor with size parameters</s>
    * <s>Remove constructor with size parameters</s>
    Line 21: Line 21:
    * <s>loadDefaultSetup protected and loadDefaultLayout</s>
    * <s>loadDefaultSetup protected and loadDefaultLayout</s>
    * <s>clearApplets -> clearContainments</s>
    * <s>clearApplets -> clearContainments</s>
    * config -> private Note : config protected in containments and applets
    * <s>config -> private Note : config protected in containments and applets</s>
         Note: remaining public for now applet needs it for Applet::globalConfig()
         Note: remaining public for now applet needs it for Applet::globalConfig()
    * <s>load/save applets -> load/save Layouts</s>
    * <s>load/save applets -> load/save Layouts</s>
    * scheduleConfigSync -> private
    * <s>scheduleConfigSync -> private</s>
    * <s>addContainment not a slot ->split</s>
    * <s>addContainment not a slot ->split</s>
    ** <s>addContainment(id, delayedFlag) [private]</s>
    ** <s>addContainment(id, delayedFlag) [private]</s>
    ** <s>addContainment() [public]</s>
    ** <s>addContainment() [public]</s>
    * containmentFor screen -> Not slot move to plasmaapp
    * <s>containmentFor screen -> Not slot</s>
        Note: it belongs to DesktopCorona instead
    * <s>containments() not a slot</s>
    * <s>containments() not a slot</s>
    * <s>launchActivated -> to plasmaapp</s>
    * <s>launchActivated -> to plasmaapp</s>
    * screenOwnerChanged -> to plasmaapp
    * <s>screenOwnerChanged -> to plasmaapp</s>
         Note: it's not really desktop specific, so will stay here
         Note: it's not really desktop specific, so will stay here
    * <s>protected slots -> private </s>
    * <s>protected slots -> private </s>
    Line 65: Line 64:


    === Plasma::Applet ===
    === Plasma::Applet ===
     
    * <s>remainSquare()+setAspectRatioMode()->merge using our own aspect ratio mode enun</s>
    * category()+name()+pluginName() dies->replaced by pluginInfo()
      <!> pluginInfo() should return a hand-made KPluginInfo if id->appletDescription isValid()
    * <s>resetConfigurationObject()->pimpl</s>
    * <s>watchForMouseMove()-> ??? (the suggested registerDragHandles is a very bad name)</s>
    * <s>protected setBackgroundHints() + public backgroundHings() const</s>; <s>as a result (set)DrawStandardBackground() dies</s>
      <s>Note: being protected AppletScript can't change it anymore: AppletScript friend of Applet?</s>
    * <s>(set)configNeeded()->(is|set)ConfigurationRequired()+set  protected</s>
    * <s>shape() dies</s>
    * <s>*Opacity() dies</s>
    * <s>launchActivated()-> releaseVisualFocus()</s>
    * <s>configXml() -> configScheme()</s>
    * <s>contextActions()->const+contextualActions()</s>
    * <s>inherits QGraphicsWidget</s>
    * <s>inherits QGraphicsWidget</s>
    * <s>init()->protecgted+Corona::loadLayout() is friend</s>
    * <s>init()->protecgted+Corona::loadLayout() is friend</s>
    * (NOTE: config will remain public) <s>config() methods->protected.</s>
    * <s>config() methods->protected.</s>
    * (NOTE: globalConfig will remain public) <s>globalConfig()->protected</s>
    * <s>globalConfig()->protected</s>
         Note: It has been decided the above two functions will remain public because if an applet uses an external QWidget it will have to access the configuration of the applet
         Note: It has been decided the above two functions will remain public because if an applet uses an external QWidget it will have to access the configuration of the applet
    * configXml()->KConfigSkeletonconfigScheme()
    * <s>constraintUpdated()->constraintEvent()+protected</s>
    * <s>constraintUpdated()->constraintEvent()+protected</s>
         Note: check again corona for events like methods
         Note: check again corona for events like methods
    * formFactors()->non virtual + get the containment impl there
    * <s>formFactors()->non virtual + get the containment impl there
       ==>Plasma::Containment::Private inherits Plasma::Applet::Private
       ==>Plasma::Containment::Private inherits Plasma::Applet::Private</s>
    * location-> Plasma::Containment::Private
    * <s>location-> Plasma::Containment::Private</s>
    * remainSquare()+setAspectRatioMode()->merge using our own aspect ratio mode enun
    * <s>knownApplets()->listAppletInfo()</s>
    * <s>knownApplets()->listAppletInfo()</s>
    * <s>knownAppletsForMimetype()->listAppletInfoForMimeType()</s>
    * <s>knownAppletsForMimetype()->listAppletInfoForMimeType()</s>
    Line 88: Line 97:
       but even if it's friend of applet it does that in the dptr,
       but even if it's friend of applet it does that in the dptr,
       and this somewhere compiles somewhere not, generally not good.
       and this somewhere compiles somewhere not, generally not good.
    * category()+name()+pluginName() dies->replaced by pluginInfo()
    * <s>color() die! die! die!</s>
      <!> pluginInfo() should return a hand-made KPluginInfo if id->appletDescription isValid()
    * color() die! die! die!
    * <s>isKioskmmutable()->pimpl</s> isImmutable() setIsImmutable() and isKioskImmutable() became immutability() and setImmutability()
    * <s>isKioskmmutable()->pimpl</s> isImmutable() setIsImmutable() and isKioskImmutable() became immutability() and setImmutability()
    * (set)DrawStandardBackground()->(is|set)StandardBackgroundEnabled()
    * <s>failedtoLaunch()->hasfailedToLaunch()</s>
    * <s>failedtoLaunch()->hasfailedToLaunch()</s>
    * <s>setFailedToLaunch()->protected</s>
    * <s>setFailedToLaunch()->protected</s>
    * (set)configNeeded()->(is|set)ConfigurationRequired()+set  protected
    * <s>hasCofigurationInterface()-> const</s>
    * <s>hasCofigurationInterface()-> const</s>
    * <s>setHasConfigurationInterface()->protected</s>
    * <s>setHasConfigurationInterface()->protected</s>
    * contextActions()->const+contextualActions()
      Note: some things e.g. desktopcontainment caches the contextual
      actions in their class params, so can't be const, 3 options:
    ** leaving non const
    ** not use caching
    ** pimpl-ify
    * <s>kill *StandardBackground()+*ShadowShown()</s>
    * <s>kill *StandardBackground()+*ShadowShown()</s>
       ->enum BackgroundHint {NoBackground=0, StandardBackground, ShadowedBackground}
       ->enum BackgroundHint {NoBackground=0, StandardBackground, ShadowedBackground}
       +QFlag
       +QFlag
    protected setBackgroundHints() + public backgroundHings() const
      Note: being protected AppletScript can't change it anymore: AppletScript friend of Applet?
    * <s>set(is)containment()->set(ActsAs)Containment()+setter is protected</s>
    * <s>set(is)containment()->set(ActsAs)Containment()+setter is protected</s>
    * <s>raise()->slot+addlower()</s>
    * <s>raise()->slot+addlower()</s>
    * shape() dies
    * resetConfigurationObject()->pimpl
    * *Opacity() dies
    * <s>requestFocus() dies</s>
    * <s>requestFocus() dies</s>
    * showConfigurationInterface()->nonvirtual
    * <s>showConfigurationInterface()->nonvirtual</s>
    * createconfigurationInterface()->virtual protected, not slot
    * <s>createconfigurationInterface()->virtual protected, not slot</s>
    * <s>Flush UpdatedConstraints()->Flush PendingCostraintEvents()</s>
    * <s>Flush UpdatedConstraints()->Flush PendingCostraintEvents()</s>
        Note: constraintEvents or constraintsEvents?
    * <s>ThemeChanged()->Q_PRIVATE_SLOT</s>
        whiteboard it's not very readable
    * <s>globalName(), instanceName()->pimpl</s>
    * launchActivated()->protected, releaseVisualFocus()
    * ThemeChanged()->Q_PRIVATE_SLOT
    * globalName(), instanceName()->pimpl
    * <s>watchForFocus()->dies</s>
    * <s>watchForFocus()->dies</s>
    * watchForMouseMove()->registerDragHandles()
    * <s>needFocus()->dies</s>
    * <s>needFocus()->dies</s>
    * <s>topLeft()->dies</s>
    * <s>topLeft()->dies</s>
    * checkImmutability()->Q_PRIVATE_SLOT
    * <s>checkImmutability()->Q_PRIVATE_SLOT</s>


    === Plasma::Containments(aseigo)  ===
    === Plasma::Containments(aseigo)  ===
    * addToolBoxTool() -> addToolBoxAction(action)
    * add removeToolBoxAction(action)
    * <s>formfactor() & location() in Applet</s>
    * <s>setFormFactor() & setLocation() in Containment</s>
    * <s>StyleOption::desktop dies</s>
    * <s>StyleOption::desktop dies</s>
    * <s>List, Dict dies</s>
    * <s>List, Dict dies</s>
    Line 135: Line 129:
    * <s>setContainmentType() -> protected</s>
    * <s>setContainmentType() -> protected</s>
    * <s>setIsContainment() -> dies (meat in applet)</s>
    * <s>setIsContainment() -> dies (meat in applet)</s>
    * formfactor() & location() in Applet
    * setFormFactor() & setLocation() in Containment
    * <s>known...() -> list...()</s>
    * <s>known...() -> list...()</s>
    * <s>addApplet() -> split</s>
    * <s>addApplet() -> split</s>
    Line 142: Line 134:
    ** <s>addApplet(name, args, point, size, id, delayedinit) [private]</s>
    ** <s>addApplet(name, args, point, size, id, delayedinit) [private]</s>
    * <s>indexAt() dies, add QPointF to appletAdded()</s>
    * <s>indexAt() dies, add QPointF to appletAdded()</s>
    * setScreen() & screen() & effectiveScreenPos() -> die
    * <s>setScreen() & screen() & effectiveScreenPos() -> die</s> will remain due to PUI relevance - aseigo
    * <s>(save|load)Constraints() -> pimpl</s>
    * <s>(save|load)Constraints() -> pimpl</s>
    * <s>emitLaunchActivated() -> see Applet, maybe not public</s>
    * <s>emitLaunchActivated() -> see Applet, maybe not public</s>
    * addToolBoxTool() -> addToolBoxAction(action)
    * add removeToolBoxAction(action)
    * <s>containmentConstraingUpdated() -> pimpl</s>
    * <s>containmentConstraingUpdated() -> pimpl</s>
    * <s>add setToolBoxExpanded(bool) slot</s>
    * <s>add setToolBoxExpanded(bool) slot</s>
    Line 156: Line 146:
    * <s>toggleDesktopImmutability() -> Q_PRIVATE_SLOT</s>
    * <s>toggleDesktopImmutability() -> Q_PRIVATE_SLOT</s>
    * <s>zoomIn() -> Q_PRIVATE_SLOT</s>
    * <s>zoomIn() -> Q_PRIVATE_SLOT</s>
    * createLayout() -> dies
    * <s>createLayout() -> dies</s>
    * <s>protected slots (except dropEvent()) & private slots & methods ->pimpl & Q_PRIVATE_SLOT</s>
    * <s>protected slots (except dropEvent()) & private slots & methods ->pimpl & Q_PRIVATE_SLOT</s>


    === Plasma::DataEngine ===
    === <s>Plasma::DataEngine</s> ===
    * <s>sourceDict() -> containerDict()</s>
    * <s>sourceDict() -> containerDict()</s>
    * <s>engineName() -> name() + add protected setName()</s>
    * <s>engineName() -> name() + add protected setName()</s>
    Line 169: Line 159:
    * <s>sourceRequested -> sourceRequestEvent()</s>
    * <s>sourceRequested -> sourceRequestEvent()</s>
    * <s>updateSource() -> updateSourceEvent()</s>
    * <s>updateSource() -> updateSourceEvent()</s>
    * setData() -> SHALL NOT remove source (fix APIDOX)
    * <s>setData() -> SHALL NOT remove source (fix APIDOX)</s>
    * <s>clearData() -> removeAllData()</s>
    * <s>clearData() -> removeAllData()</s>
    * <s>setSourceLimit() -> DIE! DIE! DIE!</s>
    * <s>setSourceLimit() -> DIE! DIE! DIE!</s>
    * <s>clearSources() -> removeAllSources()</s>
    * <s>clearSources() -> removeAllSources()</s>
    * <s>checkForUpdates() -> scheduleSourcesUpdated()</s>
    * <s>checkForUpdates() -> scheduleSourcesUpdated()</s>
    * internalUpdateSource() -> Q_PRIVATE_SLOT
    * <s>internalUpdateSource() -> Q_PRIVATE_SLOT</s>


    === <s>Plasma::DataEngineManager</s> ===
    === <s>Plasma::DataEngineManager</s> ===
    Line 182: Line 172:
    * <s>(un)load()->(un)loadEngine()</s>
    * <s>(un)load()->(un)loadEngine()</s>


    === Plasma::DataContainer ===
    === <s>Plasma::DataContainer</s> ===
    * <s>clearData()->removeAllData()</s>
    * <s>clearData()->removeAllData()</s>
    * <s>checkForUpdate()->protected</s>
    * <s>checkForUpdate()->protected</s>
    Line 190: Line 180:
    * <s>checkUsage()->pimpl</s>
    * <s>checkUsage()->pimpl</s>
    * <s>connectVisualization()->not slot</s>
    * <s>connectVisualization()->not slot</s>
    * unused()->becameUnused() DONE but move to pimpl?
    * <s>unused()->becameUnused()</s>
    * requestUpdate()->updateRequested() DONE but move to pimpl?
    * <s>requestUpdate()->updateRequested()</s>


    === Plasma::View ===
    === <s>Plasma::View</s> ===
    (Rob Scheepmaker (pinda))


    * <s>(set)DrawWallpaper() -> (set|is)WallpaperEnabled()</s>
    * <s>(set)DrawWallpaper() -> (set|is)WallpaperEnabled()</s>
    * (set)Screen() -> plasma app
    * <s>(set)Screen() -> plasma app</s> going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    * (set)Desktop() -> plasma app
    * <s>(set)Desktop() -> plasma app</s> going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    * effectiveDesktop -> plasma app
    * <s>effectiveDesktop -> plasma app</s> going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    * <s>config() -> protected</s>
    * <s>config() -> protected</s>
    * <s>updateSceneRect() -> Q_PRIVATE_SLOT</s>
    * <s>updateSceneRect() -> Q_PRIVATE_SLOT</s>
    * <s>initGraphicsView() -> pimpl</s>
    * <s>initGraphicsView() -> pimpl</s>


    === Plasma::AbstractRunner (bettio) ===
    === Plasma::AbstractRunner ===
    * <s>SlowSpeed before NormalSpeed</s>
    * <s>SlowSpeed before NormalSpeed</s>
    * <s>load() -> loadAll()</s> + load()  no = QString() for whitelist
    * <s>load() -> loadAll()</s> + load()  no = QString() for whitelist
    Line 224: Line 213:
    * <s>clearMatches() -> removeAllMatches()</s>
    * <s>clearMatches() -> removeAllMatches()</s>


    === <s>Plasma::Phase -></s> Plasma::AnimationController ===
    === Plasma runner terminology ===
    * <s>Plasma::SearchContext => Plasma::RunnerContext</s>
    * <s>Plasma::SearchMatch => Plasma::QueryMatch</s>
    * <s>In the RunnerContext, RunnerResult and AbstractRunner APIs:</s>
      * search term => query
     
    === <s>Plasma::Phase -> Plasma::Animator</s> ===
    * <s>Animation merges with ElementAnimation -> AppearAnimation,DisappearAnimation,ActivateAnimation</s>
    * <s>Animation merges with ElementAnimation -> AppearAnimation,DisappearAnimation,ActivateAnimation</s>
    * <s>SlideInMovement,SlideOutMovement,FastSlideInMovement, FastSlideOutMovement</s>
    * <s>SlideInMovement,SlideOutMovement,FastSlideInMovement, FastSlideOutMovement</s>
    * <s>AnimId dies -> use int</s>
    * <s>AnimId dies -> use int</s>
    * constructor + destructor private
    * <s>constructor + destructor private</s>
    * animateItem, moveItem return int
    * <s>animateItem, moveItem return int</s>
    * add stopItemAnimation(int)
    * <s>add stopItemAnimation(int)</s>
    * <s>animationCompleted() -> animationFinished()</s>
    * <s>animationCompleted() -> animationFinished()</s>
    * <s>same for others signal</s>
    * <s>same for others signal</s>
    * protected slots -> Q_PRIVATE_SLOT
    * <s>protected slots -> Q_PRIVATE_SLOT</s>
    * init -> pimpl
    * <s>init -> pimpl</s>
    * <s>setAnimationPixmap -> setInitialPixmap</s>
    * <s>setAnimationPixmap -> setInitialPixmap</s>
    * <s>animationResult() -> currentPixmap</s>
    * <s>animationResult() -> currentPixmap</s>


    === <s>Plasma::Animator</s> ===
    === <s>Plasma::AnimationDriver</s> ===
    * <s>FramesPerSeconds -> animationFramesPerSeconds, elementAnimationFramesPerSeconds,movementAnimationFramesPerSeconds</s>
    * <s>FramesPerSeconds -> animationFramesPerSeconds, elementAnimationFramesPerSeconds,movementAnimationFramesPerSeconds</s>
    * <s>same for duration and curve</s>
    * <s>same for duration and curve</s>
    Line 247: Line 242:
    * <s>elementX -> should use progress, not frame</s>
    * <s>elementX -> should use progress, not frame</s>


    === Plasma::GlApplet ===
    === <s>Plasma::GlApplet</s> ===
    * makeCurrent -> pimpl
    * <s>makeCurrent -> pimpl</s>: actually, this is required to be public - aseigo


    === Plasma::Delegate ===
    === Plasma::Delegate ===
    (Marco Martin)
    (Marco Martin)
    * <nowiki>*</nowiki>columnType* -> device notifier
    * <nowiki>*</nowiki>columnType* -> device notifier
      Needed for the delegate to know he must draw an icon smaller than the main one
    * <s>ctor should have parent param</s>
    * <s>ctor should have parent param</s>
    * constants + protected non virtual -> pimpl
    * <s>constants ->pimpl</s>
    * <s>protected non virtual</s> -> pimpl
       The protected methods are needed only by subclasses to know where to paint so they are still needed
       The protected methods are needed only by subclasses to know where to paint so they are still needed
    * <s>setRole()->setRoleMapping(QMap) + add roleMapping() getter</s>
    * <s>setRole()->setRoleMapping(QMap) + add roleMapping() getter</s>
    * fixme: copy rhs of whiteboard (missing)
    * fixme: copy rhs of whiteboard (missing)


    === Plasma::Dialog ===
    === <s>Plasma::Dialog</s> ===
    (Ménard Alexis)
    (Ménard Alexis)
    * <s>themeUpdated -> Q_PRIVATE_SLOT</s>
    * <s>themeUpdated -> Q_PRIVATE_SLOT</s>


    === Plasma::AppletBrowserWidget ===
    === <s>Plasma::AppletBrowserWidget</s> ===
    * remove containment from constructor
    * <s>remove containment from constructor</s>
    * protected slots -> Q_PRIVATE_SLOTS
    * <s>protected slots -> Q_PRIVATE_SLOTS</s>
    * private methods -> pimpl
    * <s>private methods -> pimpl</s>


    === Plasma::AppletBrowser ===
    === <s>Plasma::AppletBrowser</s> ===
    * remove containment from constructor
    * <s>remove containment from constructor</s>


    NOTE : AppletBrowser and AppletBrowserWidget should be in different .h
    NOTE : AppletBrowser and AppletBrowserWidget should be in different .h


    === Plasma::PackageMetadata(josebur) ===
    === <s>Plasma::PackageMetadata</s> ===
    * <s>isComplete -> isValid</s>
    * <s>isComplete -> isValid</s>
    * <s>write() -> remove icon parameter</s>
    * <s>write() -> remove icon parameter</s>
    Line 283: Line 280:
    * <s>required -> isRequired()</s>
    * <s>required -> isRequired()</s>
    * (set)(Default)Mimetypes -> (set)(Default)Mimetypes ??
    * (set)(Default)Mimetypes -> (set)(Default)Mimetypes ??
    * uninstallPackage -> packageName instead of param package
    * <s>uninstallPackage -> packageName instead of param package</s>
    * pathChanged -> died
    * <s>pathChanged -> died</s>
    * Xtypes() -> XFormat()
    * Xtypes() -> XFormat()


    === Plasma::Package(josebur) ===
    === Plasma::Package(josebur) ===
    * const char * -> QString
    * const char * -> QString
    * knownPackages -> listInstalled
    * <s>knownPackages -> listInstalled</s>
    * installPackage -> install
    * <s>installPackage -> install</s> leave it
    * registerPackage -> register '''Keyword!'''
    * <s>registerPackage -> register</s> '''Keyword!''' leave it
    * createPackage -> create
    * <s>createPackage -> create</s> leave it
    * Missing uninstall
    * Missing uninstall


    === Plasma::ScriptEngine ===
    === Plasma::ScriptEngine ===
    * contructor -> protected
    * <s>contructor -> protected</s>
    * protected slots -> not slots
    * <s>protected slots -> not slots</s>
    * knowLanguages -> getScriptApis() ??
    * knowLanguages -> getScriptApis() ??
    * loadScriptEngine() -> language param become a scriptAPI
    * loadScriptEngine() -> language param become a scriptAPI
    Line 304: Line 301:
    === Plasma::AppletScript ===
    === Plasma::AppletScript ===
    * <s>contentSizeHint -> died</s>
    * <s>contentSizeHint -> died</s>
    * constraintsUpdated -> constraintsEvent
    * <s>constraintsUpdated -> constraintsEvent</s>
    * <s>contextActions -> contextualActions</s>
    * <s>contextActions -> contextualActions</s>
    * change the K_EXPORT stuff to support several plugins by lib
    * change the K_EXPORT stuff to support several plugins by lib
    Line 310: Line 307:
    === Plasma::RunnerScript ===
    === Plasma::RunnerScript ===
    * change to new runner API
    * change to new runner API
    * protected slots -> not slots
    * <s>protected slots -> not slots</s>


    === Plasma::DataEngineScript ===
    === <s>Plasma::DataEngineScript</s> ===
    * <s>Change to new API</s>
    * <s>Change to new API</s>


    == Introduction Hotpoint List ==
    == Personal Hotpoints Shared During Introductory Session ==


    * Get QScript script bindings loading
    * Get QScript script bindings loading

    Latest revision as of 18:56, 2 May 2008

    API Reviews

    Plasma::Theme

    • self() -> default()  :NOTE: default is a C++ keyword, defaultTheme() perhaps?
    • set application dies, use KGlobal instead
    • implementation note: Theme::Private::config() shouldn't open config everytime
    • image() -> imagePath()
    • colors() -> KColorScheme colorScheme()
    • color methods -> color(ColorRole) (enum)
    • font methods -> font(RontRole) (enum)
    • compositingActive -> windowTranslucencyEnabled (out of place?)
    • changed() -> themeChanged()
    • slots -> Q_PRIVATE_SLOT

    Plasma::Corona

    • Remove constructor with size parameters
    • Remove isKioskImmutable() -> protected isImmutable() setIsImmutable() and isKioskImmutable() became immutability() and setImmutability()
    • remove maxSizeHint
    • loadDefaultSetup protected and loadDefaultLayout
    • clearApplets -> clearContainments
    • config -> private Note : config protected in containments and applets
       Note: remaining public for now applet needs it for Applet::globalConfig()
    
    • load/save applets -> load/save Layouts
    • scheduleConfigSync -> private
    • addContainment not a slot ->split
      • addContainment(id, delayedFlag) [private]
      • addContainment() [public]
    • containmentFor screen -> Not slot
    • containments() not a slot
    • launchActivated -> to plasmaapp
    • screenOwnerChanged -> to plasmaapp
       Note: it's not really desktop specific, so will stay here
    
    • protected slots -> private

    Terminology : layouts = Applets + containments organization

    Plasma::Widget

    • deletion
    • Create a tooltip manager

    Plasma::Svg

    • ContentType -> kill change methods using it to take bool and change their names to containsMultiplesImages (hmm actually not understood what this means :D)
    • elementRect -> return a QRectF
    • elementExists -> hasElement
    • matrixForElement -> private
    • size -> group with resize
    • move private slots to a pimpl

    Plasma::PanelSvg

    • should inherits of Plasma::Svg
    • drawXborder -> Xborder
    • contentAtOrigin -> separate bool property
    • SvgPanel(path,parent) -> SvgPanel(parent)
    • setFile + file() -> setImagePath + imagePath() NOTE : same modification in svg and theme classes
    • setBorderFlags + borderFlags() -> setEnabledBorders + enabledBorders
    • add resize(x,y)
    • setPrefix ->setElementsPrefix
    • updateSizes() -> Q_PRIVATE_SLOT
    • pos + setPos died -> pos added to paint
    • setLocation() -> setElementPrefix(enum)
    • location() -> died

    Plasma::Applet

    • remainSquare()+setAspectRatioMode()->merge using our own aspect ratio mode enun
    • category()+name()+pluginName() dies->replaced by pluginInfo()
      <!> pluginInfo() should return a hand-made KPluginInfo if id->appletDescription isValid()
    
    • resetConfigurationObject()->pimpl
    • watchForMouseMove()-> ??? (the suggested registerDragHandles is a very bad name)
    • protected setBackgroundHints() + public backgroundHings() const; as a result (set)DrawStandardBackground() dies
      Note: being protected AppletScript can't change it anymore: AppletScript friend of Applet?
    
    • (set)configNeeded()->(is|set)ConfigurationRequired()+set protected
    • shape() dies
    • *Opacity() dies
    • launchActivated()-> releaseVisualFocus()
    • configXml() -> configScheme()
    • contextActions()->const+contextualActions()
    • inherits QGraphicsWidget
    • init()->protecgted+Corona::loadLayout() is friend
    • config() methods->protected.
    • globalConfig()->protected
        Note: It has been decided the above two functions will remain public because if an applet uses an external QWidget it will have to access the configuration of the applet
    
    • constraintUpdated()->constraintEvent()+protected
       Note: check again corona for events like methods
    
    • formFactors()->non virtual + get the containment impl there
      ==>Plasma::Containment::Private inherits Plasma::Applet::Private
    
    • location-> Plasma::Containment::Private
    • knownApplets()->listAppletInfo()
    • knownAppletsForMimetype()->listAppletInfoForMimeType()
    • knownCategories()->listCategories()

    Note: known*()->list*()

    • because of load(), ctors->protected
      Note: containment wants to construct an Applet directly when
      creates the one with the failed to launch error message,
      but even if it's friend of applet it does that in the dptr,
      and this somewhere compiles somewhere not, generally not good.
    
    • color() die! die! die!
    • isKioskmmutable()->pimpl isImmutable() setIsImmutable() and isKioskImmutable() became immutability() and setImmutability()
    • failedtoLaunch()->hasfailedToLaunch()
    • setFailedToLaunch()->protected
    • hasCofigurationInterface()-> const
    • setHasConfigurationInterface()->protected
    • kill *StandardBackground()+*ShadowShown()
     ->enum BackgroundHint {NoBackground=0, StandardBackground, ShadowedBackground}
      +QFlag
    
    • set(is)containment()->set(ActsAs)Containment()+setter is protected
    • raise()->slot+addlower()
    • requestFocus() dies
    • showConfigurationInterface()->nonvirtual
    • createconfigurationInterface()->virtual protected, not slot
    • Flush UpdatedConstraints()->Flush PendingCostraintEvents()
    • ThemeChanged()->Q_PRIVATE_SLOT
    • globalName(), instanceName()->pimpl
    • watchForFocus()->dies
    • needFocus()->dies
    • topLeft()->dies
    • checkImmutability()->Q_PRIVATE_SLOT

    Plasma::Containments(aseigo)

    • addToolBoxTool() -> addToolBoxAction(action)
    • add removeToolBoxAction(action)
    • formfactor() & location() in Applet
    • setFormFactor() & setLocation() in Containment
    • StyleOption::desktop dies
    • List, Dict dies
    • CustomContainment = 127
    • setContainmentType() -> protected
    • setIsContainment() -> dies (meat in applet)
    • known...() -> list...()
    • addApplet() -> split
      • addApplet(name, args, point, size) [public]
      • addApplet(name, args, point, size, id, delayedinit) [private]
    • indexAt() dies, add QPointF to appletAdded()
    • setScreen() & screen() & effectiveScreenPos() -> die will remain due to PUI relevance - aseigo
    • (save|load)Constraints() -> pimpl
    • emitLaunchActivated() -> see Applet, maybe not public
    • containmentConstraingUpdated() -> pimpl
    • add setToolBoxExpanded(bool) slot
    • (show|hide)ToolBox() -> (expand|unexpand)ToolBox()
    • launchActivated() -> see Applet
    • zoomIn() & zoomOut() signals -> zoomRequested(containment, direction enum)
    • showAddWidgets() -> showAddWidgetsInterface(QPointF)
    • screenChanged() -> dies (move to plasma app ScreenManager)
    • toggleDesktopImmutability() -> Q_PRIVATE_SLOT
    • zoomIn() -> Q_PRIVATE_SLOT
    • createLayout() -> dies
    • protected slots (except dropEvent()) & private slots & methods ->pimpl & Q_PRIVATE_SLOT

    Plasma::DataEngine

    • sourceDict() -> containerDict()
    • engineName() -> name() + add protected setName()
    • connectSource() -> updateInterval -> pollingInterval
    • connectAllSources() -> updateInterval -> pollingInterval
    • ref() & deref() & isUsed() -> private
    • setIcon() -> protected
    • newSource -> sourceAdded()
    • sourceRequested -> sourceRequestEvent()
    • updateSource() -> updateSourceEvent()
    • setData() -> SHALL NOT remove source (fix APIDOX)
    • clearData() -> removeAllData()
    • setSourceLimit() -> DIE! DIE! DIE!
    • clearSources() -> removeAllSources()
    • checkForUpdates() -> scheduleSourcesUpdated()
    • internalUpdateSource() -> Q_PRIVATE_SLOT

    Plasma::DataEngineManager

    • ctor+dtor private
    • knownEngines()->listAllEngines()
    • get()->engine()
    • (un)load()->(un)loadEngine()

    Plasma::DataContainer

    • clearData()->removeAllData()
    • checkForUpdate()->protected
    • int timeSinceLastUpdate()->protected+return uint
    • hasUpdates()->pimpl
    • setNeedsUpdate()->pimpl
    • checkUsage()->pimpl
    • connectVisualization()->not slot
    • unused()->becameUnused()
    • requestUpdate()->updateRequested()

    Plasma::View

    • (set)DrawWallpaper() -> (set|is)WallpaperEnabled()
    • (set)Screen() -> plasma app going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    • (set)Desktop() -> plasma app going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    • effectiveDesktop -> plasma app going to leave this here, as the screen management is not manditory, but useful for any PUI - aseigo
    • config() -> protected
    • updateSceneRect() -> Q_PRIVATE_SLOT
    • initGraphicsView() -> pimpl

    Plasma::AbstractRunner

    • SlowSpeed before NormalSpeed
    • load() -> loadAll() + load() no = QString() for whitelist
    • ctors -> protected
    • confg() -> protected
    • match() -> protected
    • (set)hasMatchOptions() -> (set)hasRunOptions() -> (set)hasExecOptions()? (proposed by bettio)
    • createMatchOptions() -> createRunOptions() -> createExecOptions()? (proposed by bettio)
    • runnerName() -> name()
    • init() -> Q_PRIVATE_SLOT

    Plasma::SearchContext

    • switch to value based for matches too
    • matchesChanged() emitted from performMatch()
    • resetSearchTerm(QString) -> reset()
    • mimetype() -> mimeType()
    • addMatchesTo() -> copyMatchesTo() (copy -> move)
    • determineType() -> pimpl
    • clearMatches() -> removeAllMatches()

    Plasma runner terminology

    • Plasma::SearchContext => Plasma::RunnerContext
    • Plasma::SearchMatch => Plasma::QueryMatch
    • In the RunnerContext, RunnerResult and AbstractRunner APIs:
     * search term => query
    

    Plasma::Phase -> Plasma::Animator

    • Animation merges with ElementAnimation -> AppearAnimation,DisappearAnimation,ActivateAnimation
    • SlideInMovement,SlideOutMovement,FastSlideInMovement, FastSlideOutMovement
    • AnimId dies -> use int
    • constructor + destructor private
    • animateItem, moveItem return int
    • add stopItemAnimation(int)
    • animationCompleted() -> animationFinished()
    • same for others signal
    • protected slots -> Q_PRIVATE_SLOT
    • init -> pimpl
    • setAnimationPixmap -> setInitialPixmap
    • animationResult() -> currentPixmap

    Plasma::AnimationDriver

    • FramesPerSeconds -> animationFramesPerSeconds, elementAnimationFramesPerSeconds,movementAnimationFramesPerSeconds
    • same for duration and curve
    • appear -> itemAppear
    • disappear -> itemDisappear
    • activate -> itemActivated
    • slide -> itemSlide
    • elementX -> should use progress, not frame

    Plasma::GlApplet

    • makeCurrent -> pimpl: actually, this is required to be public - aseigo

    Plasma::Delegate

    (Marco Martin)

    • *columnType* -> device notifier
      Needed for the delegate to know he must draw an icon smaller than the main one
    
    • ctor should have parent param
    • constants ->pimpl
    • protected non virtual -> pimpl
      The protected methods are needed only by subclasses to know where to paint so they are still needed
    
    • setRole()->setRoleMapping(QMap) + add roleMapping() getter
    • fixme: copy rhs of whiteboard (missing)

    Plasma::Dialog

    (Ménard Alexis)

    • themeUpdated -> Q_PRIVATE_SLOT

    Plasma::AppletBrowserWidget

    • remove containment from constructor
    • protected slots -> Q_PRIVATE_SLOTS
    • private methods -> pimpl

    Plasma::AppletBrowser

    • remove containment from constructor

    NOTE : AppletBrowser and AppletBrowserWidget should be in different .h

    Plasma::PackageMetadata

    • isComplete -> isValid
    • write() -> remove icon parameter

    Plasma::PackageStructure(josebur)

    • must not inherits of QObject
    • QList<char *> -> QStringList
    • const char * -> QString
    • required -> isRequired()
    • (set)(Default)Mimetypes -> (set)(Default)Mimetypes ??
    • uninstallPackage -> packageName instead of param package
    • pathChanged -> died
    • Xtypes() -> XFormat()

    Plasma::Package(josebur)

    • const char * -> QString
    • knownPackages -> listInstalled
    • installPackage -> install leave it
    • registerPackage -> register Keyword! leave it
    • createPackage -> create leave it
    • Missing uninstall

    Plasma::ScriptEngine

    • contructor -> protected
    • protected slots -> not slots
    • knowLanguages -> getScriptApis() ??
    • loadScriptEngine() -> language param become a scriptAPI
    • add version number to loading

    Plasma::AppletScript

    • contentSizeHint -> died
    • constraintsUpdated -> constraintsEvent
    • contextActions -> contextualActions
    • change the K_EXPORT stuff to support several plugins by lib

    Plasma::RunnerScript

    • change to new runner API
    • protected slots -> not slots

    Plasma::DataEngineScript

    • Change to new API

    Personal Hotpoints Shared During Introductory Session

    • Get QScript script bindings loading
    • QScript Runners
    • Cool scripting demo
    • Webkit
    • Default set of components for 4.1
    • Clock applet library (and how to do that for other categories of applets)
    • Improve existing applets
    • Write new applets and engines
    • Panel and panel config saner/pretification
    • Kickoff UI
    • Applet handles
    • WoC
    • Video in plasma
    • Extenders
    • System tray applet
    • Keyboard shortcuts
    • Plasma::Service
    • Usability review
    • Documentation
      • Runtime
      • Dev
      • User
    • Top level window physics
    • Plasmoid grouping
    • Plasma configuration UI
    • Devices
    • API review
    • Generic webservice support
    • Theming
    • KIcon issues
    • tooltip love (flickering, hiding..)
    • krunner looks
    • lock desktop work properly
    • use createConfigurationInterface
    • panel toolbox
    • panel hiding
    • twitter improvements
    • Phase::Animator
    • theme installation from files