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

List:       kde-devel
Subject:    Small patch for kjots
From:       Albert Astals Cid <tsdgeos () terra ! es>
Date:       2003-04-23 23:11:02
[Download RAW message or body]

Hi, i am new to kde developing.

Today I've downloaded some parts of the cvs and began searching for bugs.

I came across a small "bug" in kjots. When you open it from a shell you get
QLayout "unnamed" added to QButtonGroup "ButtonGroup_2", which already has a 
layout

I have a small patch that keeps the look and feel but removes that warning.

I hope someone with cvs access can upload this small patch to the cvs (I'm 
almost 100% sure it doesn't breaks anything, it's only 2 lines long ;-))

The patch is 

Index: KJotsMain.cpp
===================================================================
RCS file: /home/kde/kdeutils/kjots/KJotsMain.cpp,v
retrieving revision 1.92
diff -u -r1.92 KJotsMain.cpp
--- KJotsMain.cpp       15 Apr 2003 19:26:00 -0000      1.92
+++ KJotsMain.cpp       23 Apr 2003 22:59:21 -0000
@@ -289,8 +289,8 @@

   // set up the layout for the hotlist
   QBoxLayout* tempLayout = new QHBoxLayout(bg_top, KDialog::spacingHint());
-  bg_top_layout = new QHBoxLayout(bg_top, KDialog::marginHint(), 
KDialog::spacingHint());
-  tempLayout->addLayout(bg_top_layout);
+  bg_top_layout = new QHBoxLayout(tempLayout, KDialog::spacingHint());
+  bg_top_layout->setMargin(KDialog::marginHint());
   tempLayout->addItem(new QSpacerItem(30, 20, QSizePolicy::Expanding, 
QSizePolicy::Minimum ));

   // the subject list


 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]