Потому что переменные width и height заполняются значениями только 1 раз - в момент загрузки страницы. При этом изначальные значения соответствующих инпутов пусты, т.е. соответствуют NaN (Not a Number).
Перенесите заполнение этих переменных в начало функции.
|