no more using innerHTML. fixes #457
This commit is contained in:
parent
26dbae954e
commit
da7c288ff6
1 changed files with 32 additions and 10 deletions
|
|
@ -176,16 +176,38 @@ async function loadSpamReport(){
|
||||||
// Create a new row
|
// Create a new row
|
||||||
const row = document.createElement("tr");
|
const row = document.createElement("tr");
|
||||||
|
|
||||||
row.innerHTML = `
|
// Create and append each cell as a DOM element
|
||||||
<td>${report.headerMessageId}</td>
|
const tdHeaderMessageId = document.createElement("td");
|
||||||
<td>${new Date(report.message_date).toLocaleString()}</td>
|
tdHeaderMessageId.textContent = report.headerMessageId;
|
||||||
<td>${report.from.join(", ")}</td>
|
row.appendChild(tdHeaderMessageId);
|
||||||
<td>${report.subject.join(", ")}</td>
|
|
||||||
<td>${report.spamValue}</td>
|
const tdMessageDate = document.createElement("td");
|
||||||
<td>${report.moved?browser.i18n.getMessage("spamfilter_moved"):browser.i18n.getMessage("spamfilter_not_moved")} (${report.SpamThreshold})</td>
|
tdMessageDate.textContent = new Date(report.message_date).toLocaleString();
|
||||||
<td>${report.explanation}</td>
|
row.appendChild(tdMessageDate);
|
||||||
<td>${new Date(report.report_date).toLocaleString()}</td>
|
|
||||||
`;
|
const tdFrom = document.createElement("td");
|
||||||
|
tdFrom.textContent = Array.isArray(report.from) ? report.from.join(", ") : report.from;
|
||||||
|
row.appendChild(tdFrom);
|
||||||
|
|
||||||
|
const tdSubject = document.createElement("td");
|
||||||
|
tdSubject.textContent = Array.isArray(report.subject) ? report.subject.join(", ") : report.subject;
|
||||||
|
row.appendChild(tdSubject);
|
||||||
|
|
||||||
|
const tdSpamValue = document.createElement("td");
|
||||||
|
tdSpamValue.textContent = report.spamValue;
|
||||||
|
row.appendChild(tdSpamValue);
|
||||||
|
|
||||||
|
const tdMoved = document.createElement("td");
|
||||||
|
tdMoved.textContent = (report.moved ? browser.i18n.getMessage("spamfilter_moved") : browser.i18n.getMessage("spamfilter_not_moved")) + ` (${report.SpamThreshold})`;
|
||||||
|
row.appendChild(tdMoved);
|
||||||
|
|
||||||
|
const tdExplanation = document.createElement("td");
|
||||||
|
tdExplanation.textContent = report.explanation;
|
||||||
|
row.appendChild(tdExplanation);
|
||||||
|
|
||||||
|
const tdReportDate = document.createElement("td");
|
||||||
|
tdReportDate.textContent = new Date(report.report_date).toLocaleString();
|
||||||
|
row.appendChild(tdReportDate);
|
||||||
|
|
||||||
// Append the row to the table
|
// Append the row to the table
|
||||||
tableBody.appendChild(row);
|
tableBody.appendChild(row);
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue