Plasma/libplasma2/API Review/Corona: Difference between revisions

Revision as of 14:07, 4 March 2013

as

Status	Action	Method
DONE	Keep	explicit Corona(QObject * parent);
DONE	Keep	~Corona();
DONE	Keep	Plasma::Package package() const;
TO DO	Remove	void setAppletMimeType(const QString &mimetype);	<{{{3}}}>
TO DO	Remove	QString appletMimeType();	<{{{3}}}>
TO DO	Remove	QString defaultContainmentPlugin() const; -> will be in the package	<{{{3}}}>
DONE	Keep	QList<Containment*> containments() const;
DONE	Keep	KSharedConfig::Ptr config() const;
DONE	Rename	{{{2}}}
DONE	Keep	Containment *containmentForScreen(int screen) const;
TO DO	Remove	{{{2}}}	<{{{3}}}>
DONE	Keep	virtual int numScreens() const;
DONE	Keep	virtual QRect screenGeometry(int id) const;
DONE	Keep	virtual QRegion availableScreenRegion(int id) const;
DONE	Keep	QList<Plasma::Location> freeEdges(int screen) const;
TO DO	Remove	void updateShortcuts();-> Only used internally, don’t expose	<{{{3}}}>
TO DO	Remove	void addShortcuts(KActionCollection *newShortcuts);	<{{{3}}}>

@@ Line 1: / Line 1: @@
-= Methods =
-* Containment *containmentForScreen(int screen, int desktop = -1) const; Remove desktop?
-* Containment *containmentForScreen(int screen, int desktop, const QString &defaultPluginIfNonExistent,  const QVariantList &defaultArgs = QVariantList()); Remove desktop?
-Still necassary?
+{| cellspacing="0" cellpadding="5" border="1" style="border: 1px solid gray; border-collapse: collapse; text-align: left; width: 100%;" class="sortable"
-* void setAppletMimeType(const QString &mimetype);
+|- style="background: rgb(236, 236, 236) none repeat scroll 0% 0%; -moz-background-clip: border; -moz-background-origin: padding; -moz-background-inline-policy: continuous; white-space: nowrap;"
-* QString appletMimeType();
+as
-* void setDialogManager(AbstractDialogManager *manager); DialogManager was fow main windows of config uis. If all config is going to be qml, this could go (alternative is that we do an AbstractDialogManager that is a declarative view: possible as well).
+! Status
-* AbstractDialogManager *dialogManager();
+! Action
-* QString preferredToolBoxPlugin(const Containment::Type type) const; Toolboxes, instead of plugin may come from the only corona QML package.
+! Method
-void setPreferredToolBoxPlugin(const Containment::Type type, const QString &plugin);
-*  void setDefaultContainmentPlugin(const QString &name);  Default containment plugin also told in the one Corona QML package?
-* void requestConfigSync(); Ugly api: are requestConfigSync and requireConfigSync still required both?
-* void requireConfigSync();
-=Signals =
+{{FeatureDone|Keep| explicit Corona(QObject * parent);}}
-* void releaseVisualFocus();
+{{FeatureDone|Keep| ~Corona();}}
+{{FeatureDone|Keep| Plasma::Package package() const;}}
+{{FeatureTodo|Remove| void setAppletMimeType(const QString &mimetype);}}
+{{FeatureTodo|Remove| QString appletMimeType();}}
+{{FeatureTodo|Remove| QString defaultContainmentPlugin() const; -> will be in the package}}
+{{FeatureDone|Keep| QList<Containment*> containments() const;}}
+{{FeatureDone|Keep|KSharedConfig::Ptr config() const;}}
+{{FeatureDone|Rename|Containment *addContainment(const QString &name, const QVariantList &args = QVariantList()); ->createContainment, remove args}}
+{{FeatureDone|Keep|Containment *containmentForScreen(int screen) const;}}
+{{FeatureTodo|Remove|Containment *containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, const QVariantList &defaultArgs = QVariantList());}}
+{{FeatureDone|Keep|virtual int numScreens() const;}}
+{{FeatureDone|Keep|virtual QRect screenGeometry(int id) const;}}
+{{FeatureDone|Keep|virtual QRegion availableScreenRegion(int id) const;}}
+{{FeatureDone|Keep|QList<Plasma::Location> freeEdges(int screen) const;}}
+{{FeatureTodo|Remove| void updateShortcuts();-> Only used internally, don’t expose}}
+{{FeatureTodo|Remove| void addShortcuts(KActionCollection *newShortcuts);}}
+|}