maqetta/maqetta — comment on commit 492fc949c9c91fc7d633568ee71c752bf844ba38
File: maqetta.core.client/WebContent/davinci/html/HTMLModel.js line 840
By waynevicknair:

i'm wondering if this line should be assigning 'this.value' (instead of 'this.name') after having gone to the trouble of checking 'this.value' in the 'if' statement and considering that the variable 's' already contains 'this.name' assigned 4 lines above. wouldn't this yield 'checked="checked"'. my thinking is that it should be either a no-op in the case of "no value" attributes like 'checked' or 's += '="' + this.value + '"';' otherwise.