From 67a642c826a9ac3d0ca481eda7d40c47a3f9419c Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Fri, 1 Oct 2021 09:06:17 +0200
Subject: [PATCH] Replace a couple of `Array.prototype.forEach`-invocations
 with `for..of` instead

Given that `NodeList`s can be iterated using `for..of` we can use that instead, since it's a little bit nicer and easier to read than the `Array.prototype.forEach` format.
---
 src/display/annotation_layer.js | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/display/annotation_layer.js b/src/display/annotation_layer.js
index 4b677011a..ee1b47a72 100644
--- a/src/display/annotation_layer.js
+++ b/src/display/annotation_layer.js
@@ -1404,12 +1404,11 @@ class ChoiceWidgetAnnotationElement extends WidgetAnnotationElement {
       selectElement.addEventListener("updatefromsandbox", jsEvent => {
         const actions = {
           value(event) {
-            const options = selectElement.options;
             const value = event.detail.value;
             const values = new Set(Array.isArray(value) ? value : [value]);
-            Array.prototype.forEach.call(options, option => {
+            for (const option of selectElement.options) {
               option.selected = values.has(option.value);
-            });
+            }
             storage.setValue(id, {
               value: getValue(event, /* isExport */ true),
             });
@@ -1478,10 +1477,9 @@ class ChoiceWidgetAnnotationElement extends WidgetAnnotationElement {
           },
           indices(event) {
             const indices = new Set(event.detail.indices);
-            const options = event.target.options;
-            Array.prototype.forEach.call(options, (option, i) => {
-              option.selected = indices.has(i);
-            });
+            for (const option of event.target.options) {
+              option.selected = indices.has(option.index);
+            }
             storage.setValue(id, {
               value: getValue(event, /* isExport */ true),
             });