* Bug #8066 fixed - Add an OK button to x_choose. 51/16151/2
Ishit [Wed, 11 Mar 2015 09:11:52 +0000 (14:11 +0530)]
Change-Id: I5c2718a3a2a2684132ad6843aa950cb8e00e9ccf

scilab/CHANGES_5.5.X
scilab/modules/gui/src/java/org/scilab/modules/gui/bridge/messagebox/SwingScilabMessageBox.java
scilab/modules/gui/tests/nonreg_tests/bug_8066.tst [new file with mode: 0644]

index 7113062..4c75328 100644 (file)
@@ -48,6 +48,8 @@ Graphics Evolutions
 Scilab Bug Fixes
 ================
 
+* Bug #8066 fixed - There was no OK button in x_choose window.
+
 * Bug #12788 fixed - All graphics were displayed in red on some platforms.
 
 * Bug #12842 fixed - Scilab could not be launched on some platforms (ATOMS internal library loading problem).
index 35d7ec4..bc5d522 100644 (file)
@@ -583,12 +583,14 @@ public class SwingScilabMessageBox extends JDialog implements SimpleMessageBox,
             objs[1] = createXchooseListBox();
 
             // And now the buttons
-            buttons = new Object[1];
+            buttons = new Object[2];
             if (buttonsLabels != null) {
                 btnCancel.setText(buttonsLabels[0]);
             }
+            btnOK.addActionListener(this);
             btnCancel.addActionListener(this);
             buttons[0] = btnCancel;
+            buttons[1] = btnOK;
         } else if (scilabDialogType == X_DIALOG_TYPE) {
             // Create a MessageBox for Scilab x_dialog
 
@@ -766,6 +768,8 @@ public class SwingScilabMessageBox extends JDialog implements SimpleMessageBox,
                     }
                 }
                 userValue = ""; /* To make getValueSize return a non zero value */
+            } else if (scilabDialogType == X_CHOOSE_TYPE) {
+                selectedItem = listBox.getSelectedIndex() + 1;
             }
             selectedButton = 1;
         } else if (ae.getSource() == btnCancel) {
diff --git a/scilab/modules/gui/tests/nonreg_tests/bug_8066.tst b/scilab/modules/gui/tests/nonreg_tests/bug_8066.tst
new file mode 100644 (file)
index 0000000..8895022
--- /dev/null
@@ -0,0 +1,22 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Author: Ishit Mehta (ibm)
+// Copyright (C) 2015
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- INTERACTIVE TEST -->
+//
+// <-- Non-regression test for bug 8066 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=8066
+//
+// <-- Short Description -->
+//  the x_choose gui should change its behavior: - we should add a "OK" button
+
+n = x_choose(['item1';'item2';'item3'],['that is a comment';'for the dialog'])
+
+// Click on an item and click Ok.
+// Should return the 1-based index of that item
\ No newline at end of file