Reimplements material panel so traversal order is passed to it - cleans up traversal order code

This commit is contained in:
Remington Holder 2022-08-15 12:55:51 -04:00
parent 35fab7bbbc
commit 43bef18117
13 changed files with 30 additions and 55 deletions

View File

@ -105,11 +105,7 @@ public class BodyTubeConfig extends RocketComponentConfig {
order.add(check);
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
order.add(materialPanel.getMaterialCombo());
if (materialPanel.getFinishCombo() != null) {
order.add(materialPanel.getFinishCombo());
}
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "cell 4 0, gapleft paragraph, aligny 0%, spany");
//// General and General properties

View File

@ -183,12 +183,8 @@ public class EllipticalFinSetConfig extends FinSetConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "span, wrap");
order.add(materialPanel.getMaterialCombo());
if (materialPanel.getFinishCombo() != null) {
order.add(materialPanel.getFinishCombo());
}
panel.add(filletMaterialPanel(), "span, wrap");

View File

@ -210,12 +210,8 @@ public class FreeformFinSetConfig extends FinSetConfig {
}
{ //// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "span, wrap");
order.add(materialPanel.getMaterialCombo());
if (materialPanel.getFinishCombo() != null) {
order.add(materialPanel.getFinishCombo());
}
panel.add(filletMaterialPanel(), "span, wrap");
}

View File

@ -149,9 +149,8 @@ public class InnerTubeConfig extends RocketComponentConfig {
panel.add(new BasicSlider(m.getSliderModel(0, 0.1, 1.0)), "wmin 100lp, growx, wrap");
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "spanx 4, growx, wrap 15lp");
order.add(materialPanel.getMaterialCombo());
//// Right side of panel ----

View File

@ -150,12 +150,8 @@ public class LaunchLugConfig extends RocketComponentConfig {
});
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "span, wrap");
order.add(materialPanel.getMaterialCombo());
if (materialPanel.getFinishCombo() != null) {
order.add(materialPanel.getFinishCombo());
}
primary.add(panel, "grow");

View File

@ -15,9 +15,12 @@ import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JLabel;
import javax.swing.JPanel;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
/**
* Panel for configuring a component's material and finish properties.
@ -25,22 +28,20 @@ import java.util.Iterator;
public class MaterialPanel extends JPanel {
private static final Translator trans = Application.getTranslator();
private final JComboBox<Material> materialCombo;
private final JComboBox<ExternalComponent.Finish> finishCombo;
public MaterialPanel(RocketComponent component, OpenRocketDocument document,
Material.Type type, String materialString, String finishString, String partName) {
Material.Type type, String materialString, String finishString,
String partName, List<Component> order) {
super(new MigLayout("insets 0"));
JLabel label = new JLabel(materialString);
//// The component material affects the weight of the component.
label.setToolTipText(trans.get("RocketCompCfg.lbl.ttip.componentmaterialaffects"));
this.add(label, "spanx 4, wrap rel");
this.materialCombo = new JComboBox<>(new MaterialModel(this, component, type, partName));
JComboBox<Material> materialCombo = new JComboBox<>(new MaterialModel(this, component, type, partName));
//// The component material affects the weight of the component.
this.materialCombo.setToolTipText(trans.get("RocketCompCfg.combo.ttip.componentmaterialaffects"));
this.add(this.materialCombo, "spanx 4, growx, wrap paragraph");
materialCombo.setToolTipText(trans.get("RocketCompCfg.combo.ttip.componentmaterialaffects"));
this.add(materialCombo, "spanx 4, growx, wrap paragraph");
order.add(materialCombo);
if (component instanceof ExternalComponent) {
label = new JLabel(finishString);
@ -51,10 +52,11 @@ public class MaterialPanel extends JPanel {
label.setToolTipText(tip);
this.add(label, "spanx 4, wmin 220lp, wrap rel");
this.finishCombo = new JComboBox<ExternalComponent.Finish>(
JComboBox<ExternalComponent.Finish> finishCombo = new JComboBox<ExternalComponent.Finish>(
new EnumModel<ExternalComponent.Finish>(component, "Finish"));
this.finishCombo.setToolTipText(tip);
this.add(this.finishCombo, "spanx 4, growx, split");
finishCombo.setToolTipText(tip);
this.add(finishCombo, "spanx 4, growx, split");
order.add(finishCombo);
//// Set for all
JButton button = new SelectColorButton(trans.get("RocketCompCfg.but.Setforall"));
@ -81,28 +83,19 @@ public class MaterialPanel extends JPanel {
}
});
this.add(button, "wrap paragraph");
} else {
this.finishCombo = null;
order.add(button);
}
}
public MaterialPanel(RocketComponent component, OpenRocketDocument document,
Material.Type type, String partName) {
Material.Type type, String partName, List<Component> order) {
this(component, document, type, trans.get("RocketCompCfg.lbl.Componentmaterial"),
trans.get("RocketCompCfg.lbl.Componentfinish"), partName);
trans.get("RocketCompCfg.lbl.Componentfinish"), partName, order);
}
public MaterialPanel(RocketComponent component, OpenRocketDocument document,
Material.Type type) {
Material.Type type, List<Component> order) {
this(component, document, type, trans.get("RocketCompCfg.lbl.Componentmaterial"),
trans.get("RocketCompCfg.lbl.Componentfinish"), "Material");
}
public JComboBox<Material> getMaterialCombo() {
return materialCombo;
}
public JComboBox<ExternalComponent.Finish> getFinishCombo() {
return finishCombo;
trans.get("RocketCompCfg.lbl.Componentfinish"), "Material", order);
}
}

View File

@ -147,7 +147,7 @@ public class NoseConeConfig extends RocketComponentConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel2.add(materialPanel, "span, wrap");
panel.add(panel2, "cell 4 0, gapleft paragraph, aligny 0%, spany");

View File

@ -127,7 +127,7 @@ public class RailButtonConfig extends RocketComponentConfig {
panel.add(instanceablePanel(rbc), "span, wrap");
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel,"span, wrap");
primary.add(panel, "grow");

View File

@ -165,8 +165,7 @@ public class RingComponentConfig extends RocketComponentConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
order.add(materialPanel.getMaterialCombo());
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
if (component instanceof EngineBlock) {
final DescriptionArea desc = new DescriptionArea(6);

View File

@ -57,7 +57,7 @@ public class ShockCordConfig extends RocketComponentConfig {
// Material
//// Shock cord material:
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.LINE,
trans.get("ShockCordCfg.lbl.Shockcordmaterial"), null, "Material");
trans.get("ShockCordCfg.lbl.Shockcordmaterial"), null, "Material", order);
panel.add(materialPanel, "spanx 4, wrap");

View File

@ -178,7 +178,7 @@ public class TransitionConfig extends RocketComponentConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel2.add(materialPanel, "span, wrap");
panel.add(panel2, "cell 4 0, gapleft paragraph, aligny 0%, spany");

View File

@ -223,7 +223,7 @@ public class TrapezoidFinSetConfig extends FinSetConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "span, wrap");

View File

@ -158,7 +158,7 @@ public class TubeFinSetConfig extends RocketComponentConfig {
//// Material
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK);
MaterialPanel materialPanel = new MaterialPanel(component, document, Material.Type.BULK, order);
panel.add(materialPanel, "span, wrap");
primary.add(panel, "grow");