Misc. tidying.

DRY when creating empty editable table rows, and remove a couple of
unnecessary logging statements.
This commit is contained in:
Oliver Hamlet
2015-01-15 16:14:52 +00:00
parent 9e3ae973cb
commit 5da599b84b
3 changed files with 1 additions and 14 deletions
+1 -12
View File
@@ -252,19 +252,8 @@ html /deep/ table[is=editable-table] paper-icon-button[icon=add]:hover {
value: function(evt) {
/* Create new row. */
var table = evt.currentTarget.parentElement.parentElement.parentElement.parentElement.parentElement;
var rowTemplateId = table.getAttribute('data-template');
var content = document.querySelector('link[rel="import"][href$="editable-table.html"]');
if (content) {
content = content.import.getElementById(rowTemplateId).content;
} else {
content = document.getElementById(rowTemplateId).content;
}
var row = document.importNode(content, true);
table.tBodies[0].insertBefore(row, table.tBodies[0].lastElementChild);
/* Add deletion listener. */
table.tBodies[0].lastElementChild.previousElementSibling.getElementsByClassName('delete')[0].addEventListener('click', table.onRemoveRow, false);
table.addRow({});
}
},
@@ -368,7 +368,6 @@ loot-editor-close
for (var i = 0; i < inputs.length; ++i) {
if (!inputs[i].checkValidity()) {
isValid = false;
console.log(inputs[i]);
}
}
-1
View File
@@ -314,7 +314,6 @@ function areSettingsValid() {
for (var i = 0; i < inputs.length; ++i) {
if (!inputs[i].checkValidity()) {
return false;
console.log(inputs[i]);
}
}
return true;