Skip to content

Commit c144467

Browse files
committed
[INTERNAL] remove introduced advanced font setting (always apply)
1 parent cace461 commit c144467

File tree

4 files changed

+12
-36
lines changed

4 files changed

+12
-36
lines changed

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/CsvEditorSettingsExternalizable.java

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ public static final class OptionSet {
5050
public boolean SHOW_TABLE_EDITOR_INFO_PANEL;
5151
public boolean QUOTING_ENFORCED;
5252
public boolean FILE_END_LINE_BREAK;
53-
public boolean ADVANCED_FONT_HANDLING;
5453

5554
public OptionSet() {
5655
EditorSettingsExternalizable editorSettingsExternalizable = EditorSettingsExternalizable.getInstance();
@@ -70,7 +69,6 @@ public OptionSet() {
7069
TABLE_COLUMN_HIGHTLIGHTING = true;
7170
ZERO_BASED_COLUMN_NUMBERING = false;
7271
FILE_END_LINE_BREAK = true;
73-
ADVANCED_FONT_HANDLING = true;
7472
}
7573
}
7674

@@ -239,12 +237,4 @@ public boolean isTableAutoColumnWidthOnOpen() {
239237
public void setTableAutoColumnWidthOnOpen(boolean tableAutoColumnWidthOnOpen) {
240238
getState().TABLE_AUTO_COLUMN_WIDTH_ON_OPEN = tableAutoColumnWidthOnOpen;
241239
}
242-
243-
public boolean isAdvancedFontHandling() {
244-
return getState().ADVANCED_FONT_HANDLING;
245-
}
246-
247-
public void setAdvancedFontHandling(boolean advancedFontHandling) {
248-
getState().ADVANCED_FONT_HANDLING = advancedFontHandling;
249-
}
250240
}

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/CsvEditorSettingsProvider.form

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@
8686
</grid>
8787
</children>
8888
</grid>
89-
<grid id="3e325" layout-manager="GridLayoutManager" row-count="9" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
89+
<grid id="3e325" layout-manager="GridLayoutManager" row-count="8" column-count="1" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
9090
<margin top="10" left="10" bottom="10" right="10"/>
9191
<constraints>
9292
<grid row="3" column="0" row-span="1" col-span="1" vsize-policy="3" hsize-policy="3" anchor="0" fill="3" indent="0" use-parent-layout="false"/>
@@ -112,7 +112,7 @@
112112
</component>
113113
<component id="f25dc" class="javax.swing.JCheckBox" binding="cbTableColumnHighlighting">
114114
<constraints>
115-
<grid row="8" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
115+
<grid row="7" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
116116
</constraints>
117117
<properties>
118118
<text value="Enable column highlighting (Editor &gt; Color Scheme &gt; CSV &gt; Column Hightlighting Colors)"/>
@@ -134,14 +134,6 @@
134134
<text value="Adjust column width on open (default)"/>
135135
</properties>
136136
</component>
137-
<component id="67b12" class="javax.swing.JCheckBox" binding="cbAdvancedFontHandling">
138-
<constraints>
139-
<grid row="7" column="0" row-span="1" col-span="1" vsize-policy="0" hsize-policy="3" anchor="8" fill="0" indent="0" use-parent-layout="false"/>
140-
</constraints>
141-
<properties>
142-
<text value="Advanced font handling (might impact performance)"/>
143-
</properties>
144-
</component>
145137
<grid id="8ebb6" layout-manager="GridLayoutManager" row-count="1" column-count="3" same-size-horizontally="false" same-size-vertically="false" hgap="-1" vgap="-1">
146138
<margin top="0" left="0" bottom="0" right="0"/>
147139
<constraints>

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/CsvEditorSettingsProvider.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ public class CsvEditorSettingsProvider implements SearchableConfigurable {
3535
private JFormattedTextField tfMaxColumnWidth;
3636
private JFormattedTextField tfDefaultColumnWidth;
3737
private JCheckBox cbAdjustColumnWidthOnOpen;
38-
private JCheckBox cbAdvancedFontHandling;
3938

4039
@NotNull
4140
@Override
@@ -83,8 +82,7 @@ public boolean isModified() {
8382
isModified(cbFileEndLineBreak, csvEditorSettingsExternalizable.isFileEndLineBreak()) ||
8483
!tfMaxColumnWidth.getValue().equals(csvEditorSettingsExternalizable.getTableAutoMaxColumnWidth()) ||
8584
!tfDefaultColumnWidth.getValue().equals(csvEditorSettingsExternalizable.getTableDefaultColumnWidth()) ||
86-
isModified(cbAdjustColumnWidthOnOpen, csvEditorSettingsExternalizable.isTableAutoColumnWidthOnOpen()) ||
87-
isModified(cbAdvancedFontHandling, csvEditorSettingsExternalizable.isAdvancedFontHandling());
85+
isModified(cbAdjustColumnWidthOnOpen, csvEditorSettingsExternalizable.isTableAutoColumnWidthOnOpen());
8886
}
8987

9088
@Override
@@ -106,7 +104,6 @@ public void reset() {
106104
tfMaxColumnWidth.setValue(csvEditorSettingsExternalizable.getTableAutoMaxColumnWidth());
107105
tfDefaultColumnWidth.setValue(csvEditorSettingsExternalizable.getTableDefaultColumnWidth());
108106
cbAdjustColumnWidthOnOpen.setSelected(csvEditorSettingsExternalizable.isTableAutoColumnWidthOnOpen());
109-
cbAdvancedFontHandling.setSelected(csvEditorSettingsExternalizable.isAdvancedFontHandling());
110107
}
111108

112109
@Override
@@ -128,7 +125,6 @@ public void apply() throws ConfigurationException {
128125
csvEditorSettingsExternalizable.setTableAutoMaxColumnWidth((int) tfMaxColumnWidth.getValue());
129126
csvEditorSettingsExternalizable.setTableDefaultColumnWidth((int) tfDefaultColumnWidth.getValue());
130127
csvEditorSettingsExternalizable.setTableAutoColumnWidthOnOpen(cbAdjustColumnWidthOnOpen.isSelected());
131-
csvEditorSettingsExternalizable.setAdvancedFontHandling(cbAdvancedFontHandling.isSelected());
132128
}
133129

134130
protected void createUIComponents() {

src/main/java/net/seesharpsoft/intellij/plugins/csv/editor/table/swing/MultiLineCellRenderer.java

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,17 +129,15 @@ protected void setText(@NotNull String text) {
129129
protected Font determineFont(@NotNull String text) {
130130
Font finalFont = UIUtil.getFontWithFallback(EditorColorsManager.getInstance().getGlobalScheme().getFont(EditorFontType.PLAIN));
131131

132-
if (CsvEditorSettingsExternalizable.getInstance().isAdvancedFontHandling()) {
133-
FontFallbackIterator it = new FontFallbackIterator();
134-
it.setPreferredFont(finalFont.getFamily(), finalFont.getSize());
135-
it.setFontStyle(finalFont.getStyle());
136-
it.start(text, 0, text.length());
137-
for (; !it.atEnd(); it.advance()) {
138-
Font font = it.getFont();
139-
if (!font.getFamily().equals(finalFont.getFamily())) {
140-
finalFont = font;
141-
break;
142-
}
132+
FontFallbackIterator it = new FontFallbackIterator();
133+
it.setPreferredFont(finalFont.getFamily(), finalFont.getSize());
134+
it.setFontStyle(finalFont.getStyle());
135+
it.start(text, 0, text.length());
136+
for (; !it.atEnd(); it.advance()) {
137+
Font font = it.getFont();
138+
if (!font.getFamily().equals(finalFont.getFamily())) {
139+
finalFont = font;
140+
break;
143141
}
144142
}
145143

0 commit comments

Comments
 (0)