i technically prefer what you had over the onChange attribute, but this is ok.