hongshaorou/openrocket
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Bugfix] Changed Behavior In motor choosing dialog

Browse Source 
-When checking: "Limit motor length to mount length"
    -prevents other sliders from modifying value
This commit is contained in:
Daniel_M_Williams 2015-10-27 12:15:33 -04:00
parent 168f889446
commit c7c250ba8d
2 changed files with 31 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
swing/src/net/sf/openrocket/gui/dialogs/motor/thrustcurve/MotorFilterPanel.java
View File

@ -26,6 +26,7 @@ import com.itextpdf.text.Font;
import net.miginfocom.swing.MigLayout; import net.miginfocom.swing.MigLayout;
import net.sf.openrocket.gui.SpinnerEditor; import net.sf.openrocket.gui.SpinnerEditor;
import net.sf.openrocket.gui.adaptors.BooleanModel;
import net.sf.openrocket.gui.adaptors.DoubleModel; import net.sf.openrocket.gui.adaptors.DoubleModel;
import net.sf.openrocket.gui.components.UnitSelector; import net.sf.openrocket.gui.components.UnitSelector;
import net.sf.openrocket.gui.util.CheckList; import net.sf.openrocket.gui.util.CheckList;
@ -40,10 +41,11 @@ import net.sf.openrocket.startup.Application;
import net.sf.openrocket.unit.UnitGroup; import net.sf.openrocket.unit.UnitGroup;
public abstract class MotorFilterPanel extends JPanel { public abstract class MotorFilterPanel extends JPanel {
private static final long serialVersionUID = -2068101000195158181L;
private static final Translator trans = Application.getTranslator(); private static final Translator trans = Application.getTranslator();
private static Hashtable diameterLabels = new Hashtable(); private static Hashtable<Integer,JLabel> diameterLabels = new Hashtable<Integer,JLabel>();
private static double[] diameterValues = new double[] { private static double[] diameterValues = new double[] {
0, 0,
.013, .013,
@ -66,7 +68,7 @@ public abstract class MotorFilterPanel extends JPanel {
} }
} }
private static Hashtable impulseLabels = new Hashtable(); private static Hashtable<Integer,JLabel> impulseLabels = new Hashtable<Integer,JLabel>();
static { static {
int i =0; int i =0;
for( ImpulseClass impulseClass : ImpulseClass.values() ) { for( ImpulseClass impulseClass : ImpulseClass.values() ) {
@ -79,8 +81,9 @@ public abstract class MotorFilterPanel extends JPanel {
private final MotorRowFilter filter; private final MotorRowFilter filter;
private final JCheckBox limitLengthCheckBox; private final JCheckBox limitByLengthCheckBox;
private boolean limitLength = false; //private final BooleanModel limitByLengthModel;
//private boolean limitLength = false;
private Double mountLength = null; private Double mountLength = null;
private final JCheckBox limitDiameterCheckBox; private final JCheckBox limitDiameterCheckBox;
@ -99,7 +102,7 @@ public abstract class MotorFilterPanel extends JPanel {
List<Manufacturer> unselectedManusFromPreferences = ((SwingPreferences) Application.getPreferences()).getExcludedMotorManufacturers(); List<Manufacturer> unselectedManusFromPreferences = ((SwingPreferences) Application.getPreferences()).getExcludedMotorManufacturers();
filter.setExcludedManufacturers(unselectedManusFromPreferences); filter.setExcludedManufacturers(unselectedManusFromPreferences);
limitLength = ((SwingPreferences) Application.getPreferences()).getBoolean("motorFilterLimitLength", false); boolean limitByLengthPref = ((SwingPreferences) Application.getPreferences()).getBoolean("motorFilterLimitLength", false);
limitDiameter = ((SwingPreferences) Application.getPreferences()).getBoolean("motorFilterLimitDiameter", false); limitDiameter = ((SwingPreferences) Application.getPreferences()).getBoolean("motorFilterLimitDiameter", false);
//// Hide used motor files //// Hide used motor files
@ -261,21 +264,24 @@ public abstract class MotorFilterPanel extends JPanel {
sub.add( diameterSlider, "growx, wrap"); sub.add( diameterSlider, "growx, wrap");
} }
{ { // length selection
sub.add( new JLabel(trans.get("TCMotorSelPan.Length")), "split 2, wrap"); sub.add( new JLabel(trans.get("TCMotorSelPan.Length")), "split 2, wrap");
limitLengthCheckBox = new JCheckBox( trans.get("TCMotorSelPan.checkbox.limitlength")); final BooleanModel limitByLengthModel = new BooleanModel(limitByLengthPref);
GUIUtil.changeFontSize(limitLengthCheckBox, -1); limitByLengthCheckBox = new JCheckBox( limitByLengthModel );
limitLengthCheckBox.setSelected(limitLength); limitByLengthCheckBox.setText( trans.get("TCMotorSelPan.checkbox.limitlength"));
limitLengthCheckBox.addActionListener(new ActionListener() { GUIUtil.changeFontSize(limitByLengthCheckBox, -1);
limitByLengthCheckBox.addActionListener(new ActionListener() {
@Override @Override
public void actionPerformed(ActionEvent e) { public void actionPerformed(ActionEvent e) {
limitLength = limitLengthCheckBox.isSelected(); //boolean limitByLength = limitByLengthCheckBox.isSelected();
MotorFilterPanel.this.setLimitLength(); MotorFilterPanel.this.setLimitLength();
onSelectionChanged(); onSelectionChanged();
} }
}); });
sub.add( limitLengthCheckBox, "gapleft para, spanx, growx, wrap" ); sub.add( limitByLengthCheckBox, "gapleft para, spanx, growx, wrap" );
final DoubleModel minimumLength = new DoubleModel(filter, "MinimumLength", UnitGroup.UNITS_MOTOR_DIMENSIONS, 0); final DoubleModel minimumLength = new DoubleModel(filter, "MinimumLength", UnitGroup.UNITS_MOTOR_DIMENSIONS, 0);
@ -290,7 +296,7 @@ public abstract class MotorFilterPanel extends JPanel {
} }
}); });
sub.add(spin, "split 5, growx"); sub.add(spin, "split 5, growx");
limitByLengthModel.addEnableComponent(spin,false);
sub.add(new UnitSelector(minimumLength), ""); sub.add(new UnitSelector(minimumLength), "");
spin = new JSpinner(maximumLength.getSpinnerModel()); spin = new JSpinner(maximumLength.getSpinnerModel());
@ -302,7 +308,7 @@ public abstract class MotorFilterPanel extends JPanel {
} }
}); });
sub.add(spin, "growx"); sub.add(spin, "growx");
limitByLengthModel.addEnableComponent(spin,false);
sub.add(new UnitSelector(maximumLength), "wrap"); sub.add(new UnitSelector(maximumLength), "wrap");
lengthSlider = new MultiSlider(MultiSlider.HORIZONTAL,0, 1000, 0, 1000); lengthSlider = new MultiSlider(MultiSlider.HORIZONTAL,0, 1000, 0, 1000);
@ -313,6 +319,7 @@ public abstract class MotorFilterPanel extends JPanel {
lengthSlider.addChangeListener( new ChangeListener() { lengthSlider.addChangeListener( new ChangeListener() {
@Override @Override
public void stateChanged(ChangeEvent e) { public void stateChanged(ChangeEvent e) {
int minLength = lengthSlider.getValueAt(0); int minLength = lengthSlider.getValueAt(0);
minimumLength.setValue(minLength/1000.0); minimumLength.setValue(minLength/1000.0);
int maxLength = lengthSlider.getValueAt(1); int maxLength = lengthSlider.getValueAt(1);
@ -322,7 +329,7 @@ public abstract class MotorFilterPanel extends JPanel {
}); });
sub.add( lengthSlider, "growx,wrap"); sub.add( lengthSlider, "growx,wrap");
limitByLengthModel.addEnableComponent(lengthSlider,false);
} }
this.add(sub, "grow,wrap"); this.add(sub, "grow,wrap");
@ -348,8 +355,9 @@ public abstract class MotorFilterPanel extends JPanel {
} }
private void setLimitLength( ) { private void setLimitLength( ) {
((SwingPreferences) Application.getPreferences()).putBoolean("motorFilterLimitLength", limitLength); boolean limitByLength = limitByLengthCheckBox.isSelected();
if ( mountLength != null & limitLength ) { ((SwingPreferences) Application.getPreferences()).putBoolean("motorFilterLimitLength", limitByLength);
if ( mountLength != null & limitByLength ) {
lengthSlider.setValueAt(1, (int) Math.min(1000,Math.round(1000*mountLength))); lengthSlider.setValueAt(1, (int) Math.min(1000,Math.round(1000*mountLength)));
} }
} }

11
swing/src/net/sf/openrocket/gui/dialogs/motor/thrustcurve/ThrustCurveMotorSelectionPanel.java
View File

@ -311,14 +311,15 @@ public class ThrustCurveMotorSelectionPanel extends JPanel implements MotorSelec
} }
public void setMotorMountAndConfig( final FlightConfigurationID _fcid, MotorMount _mount ) { public void setMotorMountAndConfig( final FlightConfigurationID _fcid, MotorMount mountToEdit ) {
if ( null == _fcid ){ if ( null == _fcid ){
throw new NullPointerException(" attempted to set mount with a null FCID. bug. "); throw new NullPointerException(" attempted to set mount with a null FCID. bug. ");
}else if ( null == _mount ){ }else if ( null == mountToEdit ){
throw new NullPointerException(" attempted to set mount with a null mount. bug. "); throw new NullPointerException(" attempted to set mount with a null mount. bug. ");
} }
motorFilterPanel.setMotorMount(mountToEdit);
MotorInstance curMotorInstance = _mount.getMotorInstance(_fcid); MotorInstance curMotorInstance = mountToEdit.getMotorInstance(_fcid);
selectedMotor = null; selectedMotor = null;
selectedMotorSet = null; selectedMotorSet = null;
selectedDelay = 0; selectedDelay = 0;
@ -341,11 +342,9 @@ public class ThrustCurveMotorSelectionPanel extends JPanel implements MotorSelec
} }
select(motorToSelect); select(motorToSelect);
MotorMount mount = curMotorInstance.getMount();
//? have we added this motor to the given mount?
motorFilterPanel.setMotorMount(mount);
} }
motorFilterPanel.setMotorMount(mountToEdit);
scrollSelectionVisible(); scrollSelectionVisible();
} }