Review Request for KDecoration
Albert Astals Cid
aacid at kde.org
Mon Nov 3 21:29:22 GMT 2014
El Divendres, 31 d'octubre de 2014, a les 08:22:53, Martin Gräßlin va
> Hi KDE core developers,
> today I want to start the review process for the new KDecoration library.
> This library is intended to replace the window decoration library used by
> KWin and addresses some of the shortcomings we hit with the existing
> The library is a tier 1 in frameworks speak. It consists of two parts:
> * a public library to create decorations
> * a private library to create the backend (used by KWin)
I had a look at kdecoration, i could not find anything big, so i'm just
commenting on minor stuff
Some of your classes use int for positioning e.g. DecorationShadow, but other
use real like DecorationButtonGroup. Is that on purpose or a bug?
Some of your classes use QScopedPointer<Private> d; and other
std::unique_ptr<DecorationSettingsPrivate> d; Why?
If you change
void addButton(QPointer<DecorationButton> button);
void addButton(const QPointer<DecorationButton> &button);
You'll save a copy constructor of QPointer
QList<QPointer<DecorationButton> > buttons() const;
afaik if you're doing C++11 you can type
QList<QPointer<DecorationButton>> buttons() const;
> After the review the repository will be moved under KDE/Workspace and see
> its first release in Plasma 5.2. In theory it would also be a fit for
> frameworks, but KDecoration uses C++11 features not allowed in frameworks.
> If you want to give the whole thing a try you need some further development
> branches and repositories:
> * kdecoration-viewer provides a small tool to preview a decoration. Usage:
> #> kdecorationviewer plugin [theme]
> * Breeze  provides a new KDecoration theme. Once installed one can use it
> in kdecoration-viewer:
> #> kdecorationviewer org.kde.breeze
> * KWin  is ported to use the new KDecoration library and tries to load
> org.kde.breeze by default. Also the Aurorae theme engine is ported and the
> configuration module. You can switch the decoration used by KWin in
> What's still missing:
> * A window tab API. I have some ideas on how to do it, but want to wait for
> the first drafts for DWD to not stupidly block some possibilities. My ideas
> on how to integrate window tabs can be done in a ABI-stable way for the
> public library.
> * Better support for OpenGL based decorations (e.g. QtQuick). This needs to
> wait till Qt 5.4. Qt 5.4 will finally allow to share with non QOpenGLContext
> and that will hopefully allow to make better use. Once I have a Qt 5.4
> setup and experimented with it I'll draft an API for this. Most likely this
> will end in a specific subclass exposing new virtuals.
> * Various pieces in KWin, especially in the configuration area. Help is more
> than welcome :-)
> Best Regards
> Martin Gräßlin
>  kde:kdecoration
>  kde:kdecoration-viewer
>  kde:breeze (branch graesslin/kdecoration2)
>  kde:kwin (branch graesslin/kdecoration2)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: This is a digitally signed message part.
More information about the kde-core-devel