diff --git a/Tools/ArdupilotMegaPlanner/CurrentState.cs b/Tools/ArdupilotMegaPlanner/CurrentState.cs index ffd2da68ef92dcd414398716efac737717c8d530..a185006355f6b26a3f530c35a5c4441b0ef295e8 100644 --- a/Tools/ArdupilotMegaPlanner/CurrentState.cs +++ b/Tools/ArdupilotMegaPlanner/CurrentState.cs @@ -535,9 +535,6 @@ namespace ArdupilotMega case (byte)(100 + Common.ac2modes.APPROACH): mode = EnumTranslator.GetDisplayText(Common.ac2modes.APPROACH); break; - case (byte)(100 + Common.ac2modes.APPROACH): - mode = "APPROACH"; - break; case (byte)(100 + Common.ac2modes.POSITION): mode = EnumTranslator.GetDisplayText(Common.ac2modes.POSITION); break; diff --git a/Tools/ArdupilotMegaPlanner/GCSViews/ConfigurationView/ConfigFlightModes.cs b/Tools/ArdupilotMegaPlanner/GCSViews/ConfigurationView/ConfigFlightModes.cs index 863b3b996457eb281375bc31c021a2e8f01dc3e7..753e93c3b9abcda890f5086a26704b37cc099ccc 100644 --- a/Tools/ArdupilotMegaPlanner/GCSViews/ConfigurationView/ConfigFlightModes.cs +++ b/Tools/ArdupilotMegaPlanner/GCSViews/ConfigurationView/ConfigFlightModes.cs @@ -159,38 +159,19 @@ namespace ArdupilotMega.GCSViews.ConfigurationView CB_simple5.Visible = false; CB_simple6.Visible = false; - CMB_fmode1.Items.Clear(); - CMB_fmode2.Items.Clear(); - CMB_fmode3.Items.Clear(); - CMB_fmode4.Items.Clear(); - CMB_fmode5.Items.Clear(); - CMB_fmode6.Items.Clear(); - - var flightModes = EnumTranslator.Translate<Common.apmmodes>(); + var flightModes = EnumTranslator.Translate<Common.apmmodes>().Select(x => x.Value).OrderBy(x => x).ToList(); CMB_fmode1.DataSource = flightModes; - CMB_fmode1.ValueMember = "Value"; - CMB_fmode1.DisplayMember = "Value"; CMB_fmode2.DataSource = flightModes; - CMB_fmode2.ValueMember = "Value"; - CMB_fmode2.DisplayMember = "Value"; CMB_fmode3.DataSource = flightModes; - CMB_fmode3.ValueMember = "Value"; - CMB_fmode3.DisplayMember = "Value"; CMB_fmode4.DataSource = flightModes; - CMB_fmode4.ValueMember = "Value"; - CMB_fmode4.DisplayMember = "Value"; CMB_fmode5.DataSource = flightModes; - CMB_fmode5.ValueMember = "Value"; - CMB_fmode5.DisplayMember = "Value"; CMB_fmode6.DataSource = flightModes; - CMB_fmode6.ValueMember = "Value"; - CMB_fmode6.DisplayMember = "Value"; try { @@ -206,38 +187,19 @@ namespace ArdupilotMega.GCSViews.ConfigurationView } if (MainV2.cs.firmware == MainV2.Firmwares.ArduCopter2) // ac2 { - CMB_fmode1.Items.Clear(); - CMB_fmode2.Items.Clear(); - CMB_fmode3.Items.Clear(); - CMB_fmode4.Items.Clear(); - CMB_fmode5.Items.Clear(); - CMB_fmode6.Items.Clear(); - - var flightModes = EnumTranslator.Translate<Common.apmmodes>(); + var flightModes = EnumTranslator.Translate<Common.ac2modes>().Select(x => x.Value).OrderBy(x => x).ToList(); CMB_fmode1.DataSource = flightModes; - CMB_fmode1.ValueMember = "Value"; - CMB_fmode1.DisplayMember = "Value"; CMB_fmode2.DataSource = flightModes; - CMB_fmode2.ValueMember = "Value"; - CMB_fmode2.DisplayMember = "Value"; CMB_fmode3.DataSource = flightModes; - CMB_fmode3.ValueMember = "Value"; - CMB_fmode3.DisplayMember = "Value"; CMB_fmode4.DataSource = flightModes; - CMB_fmode4.ValueMember = "Value"; - CMB_fmode4.DisplayMember = "Value"; CMB_fmode5.DataSource = flightModes; - CMB_fmode5.ValueMember = "Value"; - CMB_fmode5.DisplayMember = "Value"; CMB_fmode6.DataSource = flightModes; - CMB_fmode6.ValueMember = "Value"; - CMB_fmode6.DisplayMember = "Value"; try {