Bug 13437 fixed: axis drawn by drawxis use different font 33/16233/3
Calixte DENIZET [Tue, 24 Mar 2015 14:35:27 +0000 (15:35 +0100)]
Change-Id: I8c0ec69f38686b104ee706b514d21a7d8362b313

scilab/CHANGES_5.5.X
scilab/modules/graphic_objects/src/java/org/scilab/modules/graphic_objects/builder/Builder.java
scilab/modules/graphics/sci_gateway/c/sci_drawaxis.c
scilab/modules/graphics/tests/nonreg_tests/bug_13437.dia.ref [new file with mode: 0644]
scilab/modules/graphics/tests/nonreg_tests/bug_13437.tst [new file with mode: 0644]

index bef9ce3..96f2cfb 100644 (file)
@@ -66,6 +66,8 @@ Scilab Bug Fixes
 
 * Bug #13372 fixed - xsetm should have been removed in Scilab 5.5.0.
 
+* Bug #13437 fixed - axis drawn by drawaxis used different font than those drawn by plot.
+
 * Bug #13462 fixed - Low levels functions modified bounds even with auto_scale="off".
 
 * Bug #13494 fixed - Wrong vertical range of plot3d and plot3d1 fixed.
index 2e48622..d2f5561 100644 (file)
@@ -522,8 +522,18 @@ public final class Builder {
         cloneFontContext(parent, iAxis);
 
         Font font = axis.getFont();
-        font.setSize(new Double(fontSize));
-        font.setColor(textColor);
+        if (fontSize == -1) {
+            font.setSize(axes.getFontSize());
+        } else {
+            font.setSize(new Double(fontSize));
+        }
+        if (textColor == -1) {
+            font.setColor(axes.getFontColor());
+        } else {
+            font.setColor(textColor);
+        }
+        font.setStyle(axes.getFontStyle());
+
         axis.setTicksColor(ticsColor);
 
         controller.objectCreated(iAxis);
index 65bea94..3e393ef 100644 (file)
@@ -62,7 +62,7 @@ int sci_drawaxis(char *fname, unsigned long fname_len)
     int iSubwinUID = 0;
     int minrhs = -1, maxrhs = 0, minlhs = 0, maxlhs = 1, nopt = 0;
     char dir = 'l', *format = NULL, tics = 'v', **val = NULL;
-    int fontsize = 0, sub_int = 2, seg_flag = 1, textcolor = -1, ticscolor = -1;
+    int fontsize = -1, sub_int = 2, seg_flag = 1, textcolor = -1, ticscolor = -1;
     double *x = NULL, *y = NULL;
     int nx = 0, ny = 0, ntics;
     int nb_tics_labels = -1;
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13437.dia.ref b/scilab/modules/graphics/tests/nonreg_tests/bug_13437.dia.ref
new file mode 100644 (file)
index 0000000..8993702
--- /dev/null
@@ -0,0 +1,20 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2015 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+// <-- TEST WITH GRAPHIC -->
+// <-- Non-regression test for bug 13437 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13437
+//
+// <-- Short Description -->
+// axis drawn by drawxis use different font
+plot();
+a = gca();
+axis = drawaxis(y=-1, x=1:.1:2, dir="u");
+assert_checkequal(a.font_size, axis.labels_font_size);
+assert_checkequal(a.font_style, axis.labels_font_style);
+assert_checkequal(a.font_color, axis.labels_font_color);
diff --git a/scilab/modules/graphics/tests/nonreg_tests/bug_13437.tst b/scilab/modules/graphics/tests/nonreg_tests/bug_13437.tst
new file mode 100644 (file)
index 0000000..04b6542
--- /dev/null
@@ -0,0 +1,23 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2015 - Scilab Enterprises - Calixte DENIZET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+
+// <-- TEST WITH GRAPHIC -->
+
+// <-- Non-regression test for bug 13437 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/13437
+//
+// <-- Short Description -->
+// axis drawn by drawxis use different font
+
+plot();
+a = gca();
+axis = drawaxis(y=-1, x=1:.1:2, dir="u");
+assert_checkequal(a.font_size, axis.labels_font_size);
+assert_checkequal(a.font_style, axis.labels_font_style);
+assert_checkequal(a.font_color, axis.labels_font_color);
\ No newline at end of file