[Bug 61118] Multiple Close buttons on EstateDetails

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
     
http://bugs.kde.org/show_bug.cgi?id=61118     
[email protected] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From [email protected]  2003-07-14 00:31 -------
Subject: KDE_3_1_BRANCH: kdegames/atlantik

CVS commit by kaper: 

backport for fix bug #61118: Multiple Close buttons

CCMAIL: [email protected]


  M +1 -0      ChangeLog   1.121.4.21
  M +1 -0      TODO   1.144.4.4
  M +7 -5      libatlantikui/estatedetails.cpp   1.12.4.5


--- kdegames/atlantik/ChangeLog  #1.121.4.20:1.121.4.21
@@ -14,4 +14,5 @@
 - sort user column in server list by number, not alphabetically
 - show non-game users when not in a game
+- prevent double close buttons on estate details
 
 0.5.2

--- kdegames/atlantik/TODO  #1.144.4.3:1.144.4.4
@@ -17,4 +17,5 @@
 - Update messages views after resize.
 - all player/core dependencies in Board should only be there in Mode::Play
+- don't display trade results over auction, but stack it behind it in the queue
 
 For 0.6.0

--- kdegames/atlantik/libatlantikui/estatedetails.cpp  #1.12.4.4:1.12.4.5
@@ -223,4 +223,6 @@ void EstateDetails::addButton(QString co
 void EstateDetails::addCloseButton()
 {
+        if (!m_closeButton)
+        {
         m_closeButton = new KPushButton(i18n("Close"), this);
         m_buttonBox->addWidget(m_closeButton);
@@ -226,6 +228,6 @@ void EstateDetails::addCloseButton()
         m_buttonBox->addWidget(m_closeButton);
         m_closeButton->show();
-
         connect(m_closeButton, SIGNAL(pressed()), this, SIGNAL(buttonClose()));
+        }
 }