diff --git a/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPane.java b/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPane.java index 3ac7f116e..7d38d4d39 100644 --- a/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPane.java +++ b/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPane.java @@ -34,11 +34,10 @@ public class ChartTextAttrPane extends BasicPane { private UIToggleButton bold; private UIToggleButton italic; private UIColorButton fontColor; - public static Integer[] Font_Sizes = new Integer[FONT_END - FONT_START + 1]; - + public static Integer[] FONT_SIZES = new Integer[FONT_END - FONT_START + 1]; static { for (int i = FONT_START; i <= FONT_END; i++) { - Font_Sizes[i - FONT_START] = i; + FONT_SIZES[i - FONT_START] = i; } } @@ -114,7 +113,7 @@ public class ChartTextAttrPane extends BasicPane { fontNameComboBox.setSelectedItem(frFont.getFamily()); bold.setSelected(frFont.isBold()); italic.setSelected(frFont.isItalic()); - setFontSize(frFont); + populateFontSize(frFont); if (fontColor != null) { fontColor.setColor(frFont.getForeground()); } @@ -124,7 +123,7 @@ public class ChartTextAttrPane extends BasicPane { registerAllComboBoxListener(listener); } - protected void setFontSize(FRFont frFont) { + protected void populateFontSize(FRFont frFont) { if (fontSizeComboBox != null) { fontSizeComboBox.setSelectedItem(frFont.getSize()); } @@ -177,7 +176,7 @@ public class ChartTextAttrPane extends BasicPane { } protected Object[] getFontSizeComboBoxModel() { - return Font_Sizes; + return FONT_SIZES; } protected void initState() { diff --git a/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPaneWithAuto.java b/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPaneWithAuto.java index 79863a24c..6061ed18b 100644 --- a/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPaneWithAuto.java +++ b/designer-chart/src/main/java/com/fr/design/mainframe/chart/gui/style/ChartTextAttrPaneWithAuto.java @@ -13,6 +13,14 @@ public class ChartTextAttrPaneWithAuto extends ChartTextAttrPane { private static final String AUTO = Toolkit.i18nText("Fine-Design_Basic_ChartF_Auto"); private boolean isFontSizeAuto = false; private boolean isColorAuto = false; + public static String[] FONT_SIZES_WITH_AUTO = new String[FONT_END - FONT_START + 2]; + static { + FONT_SIZES_WITH_AUTO[0] = AUTO; + + for (int i = 1; i < FONT_SIZES_WITH_AUTO.length; i++) { + FONT_SIZES_WITH_AUTO[i] = FONT_START + i - 1 + ""; + } + } public ChartTextAttrPaneWithAuto() { super(); @@ -31,19 +39,7 @@ public class ChartTextAttrPaneWithAuto extends ChartTextAttrPane { } protected Object[] getFontSizeComboBoxModel() { - if (isFontSizeAuto) { - String[] fontSizes = new String[FONT_END - FONT_START + 2]; - - fontSizes[0] = AUTO; - - for (int i = 1; i < fontSizes.length; i++) { - fontSizes[i] = FONT_START + i + ""; - } - - return fontSizes; - } - - return super.getFontSizeComboBoxModel(); + return isFontSizeAuto ? FONT_SIZES_WITH_AUTO : FONT_SIZES; } protected float getFontSize() { @@ -54,7 +50,7 @@ public class ChartTextAttrPaneWithAuto extends ChartTextAttrPane { return Float.parseFloat(GeneralUtils.objectToString(getFontSizeComboBox().getSelectedItem())); } - protected void setFontSize(FRFont frFont) { + protected void populateFontSize(FRFont frFont) { if (getFontSizeComboBox() != null && isFontSizeAuto) { if (frFont.getSize() == ChartConstants.AUTO_FONT_SIZE) { getFontSizeComboBox().setSelectedItem(AUTO);