D10170: Added optional transparency/blur to menu frames

Hugo Pereira Da Costa noreply at phabricator.kde.org
Mon Feb 12 16:01:58 UTC 2018


hpereiradacosta added inline comments.

INLINE COMMENTS

> breezeblurhelper.cpp:45
> +        QObject(parent),
> +        _helper(helper)
> +    {

helper argument is not needed

> breezeblurhelper.h:53
> +        //! constructor
> +        BlurHelper( QObject*, Helper& );
> +

helper argument is not needed

> breezeblurhelper.h:80
> +
> +        protected Q_SLOTS:
> +

this can go.
No more slots :)

> breezeblurhelper.h:85
> +        //* helper
> +        Helper _helper;
> +

this can go

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D10170

To: anemeth, hpereiradacosta, #plasma, colomar, alake
Cc: fredrik, alake, januz, abetts, colomar, andreask, zzag, ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180212/9fbec6e4/attachment.html>


More information about the Plasma-devel mailing list