[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request 122328: Fix apply button in breeze window decoration config
From:       Pinak Ahuja <pinak.ahuja () gmail ! com>
Date:       2015-01-30 11:58:20
Message-ID: CAAbh0gD5usbCgNdiTMcx1Xiq6YAOno6wg82fYxDSYhP4ruE07A () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


I might be wrong, but shouldn't the changed signal have a bool parameter?
The changed() without a parameter refers to a slot(can we emit a slot? I'm
new to this). I didn't compile and check if the behaviour was fixed in the
master branch without this patch, I have the problem with the plasma 5.2
package shipped with arch Linux.

[Attachment #5 (text/html)]

<p dir="ltr">I might be wrong, but shouldn&#39;t the changed signal have a bool \
parameter? The changed() without a parameter refers to a slot(can we emit a slot? \
I&#39;m new to this). I didn&#39;t compile and check if the behaviour was fixed in \
the master branch without this patch, I have the problem with the plasma 5.2 package \
shipped with arch Linux.</p>



_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]