Changeset 100

Timestamp:

08/28/09 13:45:06 (12 months ago)

Author:

ibboard

Message:

• Refactor out "label with close button" creation to new method (useful for  warfoundry:ticket:125)
• Change close from deleting to removing page (fixes  warfoundry:ticket:96)

no-open-ticket

Files:

• IBBoard.GtkSharp/trunk/NotebookUtil.cs (modified) (2 diffs)

IBBoard.GtkSharp/trunk/NotebookUtil.cs

@@ -28 +28 @@
                 public static int AddPageToNotebookWithCloseButton(Notebook notebook, Widget page, string title)
                 {
+                        return notebook.AppendPage(page, CreateNotebookTabLabelWithClose(notebook, page, title)); 
+                }
+                
+                /// <summary>
+                /// Creates a widget to be used as the label for a notebook tab with text and a close button
+                /// </summary>
+                /// <param name="notebook">
+                /// The <see cref="Notebook"/> the page is on
+                /// </param>
+                /// <param name="page">
+                /// The <see cref="Widget"/> that is the content of the page
+                /// </param>
+                /// <param name="title">
+                /// The text to display on the tab
+                /// </param>
+                /// <returns>
+                /// A <see cref="Widget"/> that can be used as a tab label that contains the label with the title text and a close button
+                /// </returns>
+                public static Widget CreateNotebookTabLabelWithClose(Notebook notebook, Widget page,String title)
+                {
                         HBox hbox = new HBox();
                         hbox.PackStart(new Label(title));                       
@@ -41 +61 @@
                         close.Name = "NotebookTab.CloseButton";                 
                         close.Clicked += delegate {
-                            hbox.Destroy();
-                            page.Destroy();
+                                notebook.Remove(page);
                         };
                         hbox.PackStart(close);                  
-                        hbox.ShowAll();                 
-                        return notebook.AppendPage(page, hbox); 
+                        hbox.ShowAll(); 
+                        return hbox;
                 }
         }