вообще если понятность не очень страдает то я экономлю место по вертикали и не создаю переменные которые используются 1, 2 раза. лучше чтобы код по вертикали был компактнее.
if(!(this instanceof Imgt)) {
return new Imgt(id);
}
this.element = document.getElementById(id);
тут явно строка пустая не нужна например. сверху от условия да, чтобы не сливалось можно пропустить.
из-за таких лишних строк кроме одной ф-й ни чего на экране не видно, не видно как она связана с другим кодом.
p.s.: хз, многим покажется мелочью и мб они будут правы, но мне глаз режет как то это всё последнее время.
p.s.s: документационные комментарии все мои усилия сводят на нет