Если по логике, то в property должно храниться свойство конкретного объекта, всё остальное должно быть в переменных. Поскольку выбранный квадрат имеет мало отношения к input#image-upload, то я бы складывал в переменную. Только переименовал бы её. Когда писал код, не задумывался над именем. Ну а (function(){}()) защищает глобальную область от лишних переменных - то есть этот код будет бесконфликтно встраиваться в любой скрипт, где $ = jQuery.
Но вообще сам бы с удовольствием послушал о том, как делать "правильно".
|