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;
DONE Remove void setAppletMimeType(const QString &mimetype);
DONE Remove QString appletMimeType();
TO DO Remove QString defaultContainmentPlugin() const; -> will be in the package <{{{3}}}>
DONE Keep QList<Containment*> containments() const;
DONE Keep KSharedConfig::Ptr config() const;
IN PROGRESS Rename Containment *addContainment(const QString &name, const QVariantList &args ); ->createContainment, remove args
DONE Keep Containment *containmentForScreen(int screen) const;
DONE Remove Containment *containmentForScreen(int screen, const QString &defaultPluginIfNonExistent, const QVariantList &defaultArgs );
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;
DONE Remove void updateShortcuts();-> Only used internally, don’t expose
DONE Remove void addShortcuts(KActionCollection *newShortcuts); also remove: 
       * CoronaPrivate::actionCollections
       * "configure shortcuts" actions
       * CoronaPrivate::showShortcutConfig
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 SLOT void initializeLayout(const QString &config ); <{{{3}}}>
DONE Keep SLOT void loadLayout(const QString &config );
DONE Keep SLOT void saveLayout(const QString &config ) const;
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}}}>


Retrieved from "https://community.kde.org/index.php?title=Plasma/libplasma2/API_Review/Corona&oldid=30436"