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

List:       kde-devel
Subject:    Patch: Plasma, makes compilation possible again.
From:       "Max Blazejak" <m43ksrocks () gmail ! com>
Date:       2007-05-26 20:21:18
Message-ID: 9dd8fd8b0705261321i400972bfm797c3b26adab1ac7 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi,

this Patch fixes an error in workspace/plasma/applets/clock/clock.cpp.
The okButton is nowhere defined, and the closeButton which is created there,
is never used, so I assumed that the okButton actually should be the
closeButton and changed it that way.
Tested it, compiles fine, the clock configuration dialog works fine.
So, if there's nothing against it, it'd be great if someone could commit the
patch

Max

[Attachment #5 (text/html)]

Hi,<br><br>this Patch fixes an error in \
workspace/plasma/applets/clock/clock.cpp.<br>The okButton is nowhere defined, and the \
closeButton which is created there, is never used, so I assumed that the okButton \
actually should be the closeButton and changed it that way. <br>Tested it, compiles \
fine, the clock configuration dialog works fine.<br>So, if there&#39;s nothing \
against it, it&#39;d be great if someone could commit the patch<br><br>Max<br>


["Plasma.diff" (text/x-patch)]

Index: workspace/plasma/applets/clock/clock.cpp
===================================================================
--- workspace/plasma/applets/clock/clock.cpp	(Revision 668575)
+++ workspace/plasma/applets/clock/clock.cpp	(Arbeitskopie)
@@ -106,7 +106,7 @@
 
     if (m_dialog == 0) {
         m_dialog = new QDialog;
-        connect(okButton, SIGNAL(clicked()), m_dialog, SLOT(accept()));
+        connect(closeButton, SIGNAL(clicked()), m_dialog, SLOT(accept()));
         connect(m_showTimeString, SIGNAL(stateChanged(int)), this, SLOT(acceptedConfigDialog()));
         m_dialog->setLayout(lay);
     }


>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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

Configure | About | News | Add a list | Sponsored by KoreLogic