Fix - Logo should say "Lighting" not "Lightning" (#1510)

* Fix #1505 - Logo should say "Lighting" not "Lightning"

* Address LGTM findings

* Specify an image theme using html, as fragmented approach
This commit is contained in:
LordGrey
2022-09-05 23:53:41 +02:00
committed by GitHub
parent 2008e2c3d7
commit 9f6704118e
6 changed files with 13 additions and 6 deletions

View File

@@ -183,7 +183,7 @@ $(document).ready(function () {
// Disable or enable Delete Effect Button
$('#effectsdellist').off().on('change', function () {
var value = $(this).val();
value == null ? $('#btn_edit, #btn_delete').prop('disabled', true) : "";
value == null ? $('#btn_edit').prop('disabled', true) : $('#btn_edit').prop('disabled', false);
value.startsWith("int_") ? $('#btn_delete').prop('disabled', true) : $('#btn_delete').prop('disabled', false);
});

View File

@@ -384,7 +384,7 @@
// Check if the restore to Latin number flag is set:
if ( integer ) {
if ( parseFloat( num, 10 ) === num ) {
if ( parseFloat( num, 10 ) === num ) { //lgtm [js/superfluous-trailing-arguments]
return num;
}
@@ -405,7 +405,7 @@
}
}
return integer ? parseFloat( convertedNumber, 10 ) : convertedNumber;
return integer ? parseFloat( convertedNumber, 10 ) : convertedNumber; //lgtm [js/superfluous-trailing-arguments]
},
/**

View File

@@ -889,7 +889,11 @@ function createTableRow(list, head, align) {
if (align)
el.style.verticalAlign = "middle";
el.innerHTML = list[i];
var purifyConfig = {
ADD_TAGS: ['button'],
ADD_ATTR: ['onclick']
};
el.innerHTML = DOMPurify.sanitize(list[i], purifyConfig);
row.appendChild(el);
}
return row;