Plasma/libplasma2/API Review/Corona

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	Containment *addContainment(const QString &name, const QVariantList &args ); ->createContainment, remove args
DONE	Keep	Containment *containmentForScreen(int screen) const;
TO DO	Remove	Containment *containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, const QVariantList &defaultArgs );	<{{{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); also remove:        * CoronaPrivate::actionCollections        * "configure shortcuts" actions        * CoronaPrivate::showShortcutConfig	<{{{3}}}>
DONE	Keep	KActionCollection* actions() const;
TO DO	Remove	void setContainmentActionsDefault(Plasma::ContainmentType containmentType, QEventQString *trigger, const QString &name);	<{{{3}}}>
TO DO	Remove	QString containmentActionsDefault(ContainmentType containmentType, QEvent *triggerQString &trigger) const;	<{{{3}}}>
TO DO	Rename	QHash<QString, QString>containmentActionsDefaults(ContainmentType containmentType) const; ->defaultContainmentActions() and reads what’s in the package()	<{{{3}}}>
TO DO	Remove	void setDialogManager(AbstractDialogManager *manager); Remove AbstractDialogManager class	<{{{3}}}>
TO DO	Remove	AbstractDialogManager *dialogManager();	<{{{3}}}>
TO DO	Remove	QString preferredToolBoxPlugin(const ContainmentType type) const; -> into the package()	<{{{3}}}>
DONE	Keep	QList<Plasma::Containment *> importLayout(const KConfigGroup &config);
DONE	Keep	void exportLayout(KConfigGroup &config, QList<Containment*> containments);
TO DO	Remove	{{{2}}}	<{{{3}}}>
DONE	Keep	{{{2}}}
DONE	Keep	{{{2}}}
DONE	Keep	SLOT ImmutabilityType immutability() const;
DONE	Keep	SLOT void setImmutability(const ImmutabilityType immutable);
DONE	Keep	SLOT void requestConfigSync();
DONE	Keep	SLOT void requireConfigSync();
DONE	Keep	SIGNAL void containmentAdded(Plasma::Containment *containment);
DONE	Keep	SIGNAL void screenOwnerChanged(int wasScreen, int isScreen, Plasma::Containment *containment);
TO DO	Remove	SIGNAL void releaseVisualFocus();	<{{{3}}}>
DONE	Keep	SIGNAL void configSynced(); ->remains, still a workaround though
DONE	Keep	SIGNAL void availableScreenRegionChanged();
DONE	Keep	SIGNAL void immutabilityChanged(Plasma::ImmutabilityType immutability);
TO DO	Remove	SIGNAL void shortcutsChanged();	<{{{3}}}>
DONE	Keep	protected virtual void loadDefaultLayout();
TO DO	Remove	protected  void setPreferredToolBoxPlugin(const ContainmentType type, const QString &plugin); -> read from package()	<{{{3}}}>
TO DO	Remove	protected void setDefaultContainmentPlugin(const QString &name);-> goes in package(), will be done for containment type	<{{{3}}}>