Mercurial > repos > IBBoard.WarFoundry.GUI.WinForms
diff FrmEditUnitEquipment.cs @ 98:c7afc7824f50
Fixes #218: "Amount too high" warning can result in negative items
* Restrict min/max values to a minimum of "0" to stop negatives
* Ignore all events from widgets in EquimentAmountControl until we've finished setting it up for the new equipment item
* Fire value modified events if the value has been limited
* Remove disabling of OK button so that "Edit" dialog enables OK if amount has been limited
| author | IBBoard <dev@ibboard.co.uk> |
|---|---|
| date | Sat, 21 Nov 2009 15:33:00 +0000 |
| parents | 0bb9f40d44eb |
| children | 3e78af88ceb3 |
line wrap: on
line diff
--- a/FrmEditUnitEquipment.cs Mon Nov 16 20:56:54 2009 +0000 +++ b/FrmEditUnitEquipment.cs Sat Nov 21 15:33:00 2009 +0000 @@ -43,7 +43,6 @@ this.Text = Translation.GetTranslation("FrmEditUnitEquipment", "edit {0} for {1}", equipItem.Name, unit.Name); equipmentAmount.SetUnit(unit); equipmentAmount.SetUnitEquipmentItem(equipItem); - bttnOkay.Enabled = false; } /// <summary> @@ -187,15 +186,15 @@ this.Close(); } - private void setOkayButton() + private void SetOkayButton() { double equipAmount = equipmentAmount.EquipmentAmount; - bttnOkay.Enabled = equipAmount > 0 || equipAmount == WarFoundryCore.INFINITY || !equipItem.IsRequired; + bttnOkay.Enabled = equipAmount > 0 || !equipItem.IsRequired; } private void equipmentAmount_ValueChanged(object sender, EventArgs e) { - setOkayButton(); + SetOkayButton(); } } }