Re: [Triplea-develop] Please apply patch.

[Sean B. <sbr...@ya...>]

The bug is now closed.

Sean



________________________________
From: Chris McIntosh <cmc...@gm...>
To: Sean Bridges <sbr...@ya...>
Cc: tri...@li...
Sent: Sunday, February 1, 2009 10:32:47 AM
Subject: Re: [Triplea-develop] Please apply patch.

Its only one line I'm not sure why that makes it more crowded/disorganized. But I understand.

For what its worth I found this feature request open for the same thing: https://sourceforge.net/tracker2/index.php?func=detail&aid=1560049&group_id=44492&atid=439740

Should that be closed out if its decided not to do this?

Thanks,
Chris McIntosh


On Sun, Feb 1, 2009 at 9:03 AM, Sean Bridges <sbr...@ya...> wrote:

I'm not sure I like adding another label to the territory panel.  The panel is already pretty crowded and disorganized. 

Sean



________________________________
From: Chris McIntosh <cmc...@gm...>
To: tri...@li...
Sent: Saturday, January 31, 2009 8:08:17 PM
Subject: Re: [Triplea-develop] Please apply patch.


Oops forgot to include import in patch: 

Index: TerritoryDetailPanel.java
===================================================================
--- TerritoryDetailPanel.java    (revision 2293)
+++ TerritoryDetailPanel.java    (working copy)
@@ -15,6 +15,7 @@
 package games.strategy.triplea.ui;
 
 import games.strategy.engine.data.*;
+import games.strategy.triplea.attatchments.TerritoryAttachment;
 import games.strategy.triplea.oddsCalculator.ta.OddsCalculatorDialog;
 import games.strategy.triplea.util.*;
 import games.strategy.ui.OverlayIcon;
@@ -104,6 +105,13 @@
 
         add(new JLabel(territory.getName()));
         
+        // Add Production count for territory to panel.        
+        TerritoryAttachment ta = TerritoryAttachment.get(territory);
+        if (ta != null) 
+        {            
+            add(new JLabel("Production: " + ta.getProduction()));
+        }
+
         add(m_showOdds);
         
         Collection<Unit> unitsInTerritory;



On Sat, Jan 31, 2009 at 7:43 PM, Chris McIntosh <cmc...@gm...> wrote:

I was playing and found it odd that the Territory Detail Panel didn't have the production count in it. I know its in the status bar at the bottom. Please apply this patch to add it.

Thanks,
Chris McIntosh (compavalanche at sourceforge.net) 

Index: TerritoryDetailPanel.java
===================================================================
--- TerritoryDetailPanel.java    (revision 2293)
+++ TerritoryDetailPanel.java    (working copy)
@@ -104,6 +104,13 @@
 
         add(new JLabel(territory.getName()));
         
+        // Add Production count for territory to panel.        
+        TerritoryAttachment ta = TerritoryAttachment.get(territory);
+        if (ta != null) 
+        {            
+            add(new JLabel("Production: " + ta.getProduction()));
+        }
+
         add(m_showOdds);
         
         Collection<Unit> unitsInTerritory;