Merge lp:~simar/clicompanion/fix-742394 into lp:clicompanion

[Gursimran singh]

Fixes the bug https://bugs.launchpad.net/clicompanion/+bug/742394.
All the details of the bug is given in the launchpad bug.

How I fixed:
Added a new global variable nop (no of pages), which will keep track of how many tabs are there in the open state. Used logic to hide the tabs when it is <=1 and show when it is >1..

All changes in tabs.py..

[Duane Hinnen (duanedesign)]

bdfhjk included these changes in his merge:

lp:~bdfhjk/clicompanion/fix-739402

So I merged that branch.

Great work simar!

review: Approve

=== modified file 'clicompanionlib/tabs.py'
--- clicompanionlib/tabs.py	2011-03-21 18:04:14 +0000
+++ clicompanionlib/tabs.py	2011-03-25 12:03:30 +0000
@@ -34,6 +34,9 @@
 #definition gcp - how many pages is visible
 gcp=0;
 
+#definition nop - (no of pages) reflects no of terminal tabs left (some may be closed by the user)
+nop=0;
+
 class Tabs(object):
     '''
     add a new terminal in a tab above the current terminal
@@ -56,10 +59,12 @@
         #notebook.set_show_border(True)
         
         global gcp
+        global nop
+        nop += 1
         gcp += 1
         pagenum = ('Tab %d') % gcp
-        if gcp > 1:
-			notebook.set_show_tabs(True)
+        if nop > 1:
+            notebook.set_show_tabs(True)
         box = gtk.HBox()
         label = gtk.Label(pagenum)
         box.pack_start(label, True, True)
@@ -93,7 +98,12 @@
         ## get the page number of the tab we wanted to close
         pagenum = notebook.page_num(widget)
         ## and close it
-        notebook.remove_page(pagenum) 
+        notebook.remove_page(pagenum)
+        global nop
+        nop -= 1
+        if nop <= 1:
+            notebook.set_show_tabs(False)
+        
         
     def update_term_config(self, _vte):
         ##read config file