Skip to content

Commit 71c89d3

Browse files
Merge PR mebjas#821 from upstream
2 parents 9f564c9 + c332881 commit 71c89d3

File tree

3 files changed

+17
-17
lines changed

3 files changed

+17
-17
lines changed

src/html5-qrcode-scanner.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,7 @@ export class Html5QrcodeScanner {
270270
if (!container) {
271271
throw `HTML Element with id=${this.elementId} not found`;
272272
}
273-
container.innerHTML = "";
273+
container.textContent = "";
274274
this.createBasicLayout(container!);
275275
this.html5Qrcode = new Html5Qrcode(
276276
this.getScanRegionId(),
@@ -334,7 +334,7 @@ export class Html5QrcodeScanner {
334334
const emptyHtmlContainer = () => {
335335
const mainContainer = document.getElementById(this.elementId);
336336
if (mainContainer) {
337-
mainContainer.innerHTML = "";
337+
mainContainer.textContent = "";
338338
this.resetBasicLayout(mainContainer);
339339
}
340340
}
@@ -1044,15 +1044,15 @@ export class Html5QrcodeScanner {
10441044
this.getScanRegionId())!;
10451045

10461046
if (this.cameraScanImage) {
1047-
qrCodeScanRegion.innerHTML = "<br>";
1048-
qrCodeScanRegion.appendChild(this.cameraScanImage);
1047+
const br = document.createElement("br");
1048+
qrCodeScanRegion.replaceChildren(br, this.cameraScanImage);
10491049
return;
10501050
}
10511051

10521052
this.cameraScanImage = new Image;
10531053
this.cameraScanImage.onload = (_) => {
1054-
qrCodeScanRegion.innerHTML = "<br>";
1055-
qrCodeScanRegion.appendChild($this.cameraScanImage!);
1054+
const br = document.createElement("br");
1055+
qrCodeScanRegion.replaceChildren(br, $this.cameraScanImage!);
10561056
}
10571057
this.cameraScanImage.width = 64;
10581058
this.cameraScanImage.style.opacity = "0.8";
@@ -1066,15 +1066,15 @@ export class Html5QrcodeScanner {
10661066
this.getScanRegionId())!;
10671067

10681068
if (this.fileScanImage) {
1069-
qrCodeScanRegion.innerHTML = "<br>";
1070-
qrCodeScanRegion.appendChild(this.fileScanImage);
1069+
const br = document.createElement("br");
1070+
qrCodeScanRegion.replaceChildren(br, this.fileScanImage);
10711071
return;
10721072
}
10731073

10741074
this.fileScanImage = new Image;
10751075
this.fileScanImage.onload = (_) => {
1076-
qrCodeScanRegion.innerHTML = "<br>";
1077-
qrCodeScanRegion.appendChild($this.fileScanImage!);
1076+
const br = document.createElement("br");
1077+
qrCodeScanRegion.replaceChildren(br, $this.fileScanImage!);
10781078
}
10791079
this.fileScanImage.width = 64;
10801080
this.fileScanImage.style.opacity = "0.8";
@@ -1085,7 +1085,7 @@ export class Html5QrcodeScanner {
10851085
private clearScanRegion() {
10861086
const qrCodeScanRegion = document.getElementById(
10871087
this.getScanRegionId())!;
1088-
qrCodeScanRegion.innerHTML = "";
1088+
qrCodeScanRegion.textContent = "";
10891089
}
10901090

10911091
//#region state getters

src/html5-qrcode.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1464,7 +1464,7 @@ export class Html5Qrcode {
14641464
}
14651465
const element = document.getElementById(this.elementId);
14661466
if (element) {
1467-
element.innerHTML = "";
1467+
element.textContent = "";
14681468
}
14691469
}
14701470

tests/ui/scanner/camera-selection-ui.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe("CameraSelectionUi#create()", () => {
2222

2323
after(() => {
2424
document.body.removeChild(parentElement!);
25-
parentElement!.innerHTML = "";
25+
parentElement!.textContent = "";
2626
parentElement = undefined;
2727
});
2828

@@ -40,7 +40,7 @@ describe("CameraSelectionUi#create()", () => {
4040
});
4141

4242
it("Single cameras, creates the camera selection", () => {
43-
parentElement!.innerHTML = "";
43+
parentElement!.textContent = "";
4444
let numCameras = 1;
4545
let cameras = createCameraList(numCameras);
4646
let cameraSelectUi = CameraSelectionUi.create(parentElement!, cameras);
@@ -58,7 +58,7 @@ describe("CameraSelectionUi#create()", () => {
5858
let cameras = createCameraList(numCameras);
5959
expect(() => {
6060
let _ = CameraSelectionUi.create(parentElement!, cameras);
61-
}).to.throw();
61+
}).to.throw();
6262
});
6363
});
6464

@@ -72,7 +72,7 @@ describe("CameraSelectionUi#enable() & disable()", () => {
7272

7373
after(() => {
7474
document.body.removeChild(parentElement!);
75-
parentElement!.innerHTML = "";
75+
parentElement!.textContent = "";
7676
parentElement = undefined;
7777
});
7878

@@ -113,7 +113,7 @@ describe("CameraSelectionUi setting and getting values", () => {
113113

114114
after(() => {
115115
document.body.removeChild(parentElement!);
116-
parentElement!.innerHTML = "";
116+
parentElement!.textContent = "";
117117
parentElement = undefined;
118118
});
119119

0 commit comments

Comments
 (0)