Bug 12248 fixed: SciNotes toolbar had disappeared... 94/10494/3
Calixte DENIZET [Fri, 15 Feb 2013 13:44:33 +0000 (14:44 +0100)]
Change-Id: I0cedcab6020f4177d241ba06bd7be027f26d2175

scilab/CHANGES_5.4.X
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/tab/SwingScilabTab.java

index 1b70a25..2ec2799 100644 (file)
@@ -355,6 +355,8 @@ Bug fixes
 
 * Bug #12247 fixed - Fix a typo in some error messages.
 
+* Bug #12248 fixed - SciNotes toolbar had disappeared.
+
 * Bug #12272 fixed - Fix a presentation of Cyrillic letters on sample images of russian help page for grand function.
 
 * Bug #12284 fixed - Completion in console could to a crash (or deadlock).
index d9e53bf..3457ab6 100644 (file)
@@ -1166,13 +1166,14 @@ public class SwingScilabTab extends View implements SwingViewObject, SimpleTab,
      */
     @Override
     public void setMenuBar(MenuBar newMenuBar) {
-        if (this.menuBar != null) {
-            ((SwingScilabMenuBar) this.menuBar.getAsSimpleMenuBar()).close();
+        if (this.menuBar != newMenuBar) {
+            if (this.menuBar != null) {
+                ((SwingScilabMenuBar) this.menuBar.getAsSimpleMenuBar()).close();
+            }
+            this.menuBar = newMenuBar;
         }
-        this.menuBar = newMenuBar;
     }
 
-
     /**
      * Getter for MenuBar
      * @return MenuBar : the MenuBar associated to the Tab.
@@ -1190,10 +1191,12 @@ public class SwingScilabTab extends View implements SwingViewObject, SimpleTab,
      */
     @Override
     public void setToolBar(ToolBar newToolBar) {
-        if (this.toolBar != null) {
-            ((SwingScilabToolBar) this.toolBar.getAsSimpleToolBar()).close();
+        if (this.toolBar != newToolBar) {
+            if (this.toolBar != null) {
+                ((SwingScilabToolBar) this.toolBar.getAsSimpleToolBar()).close();
+            }
+            this.toolBar = newToolBar;
         }
-        this.toolBar = newToolBar;
     }
 
     /**