Jump to content

Plasma/libplasma2/API Review/Corona

From KDE Community Wiki
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);