I agree with moving the constraints into the model (in fact I think I may have originally proposed it ;-) - so LGTM although I'm not very familiar with the UI code these days ...