mirror of
https://github.com/mozilla/pdf.js.git
synced 2025-04-19 22:58:07 +02:00
Merge pull request #14095 from Snuffleupagus/annotationLayer-rm-forEach
Replace a couple of `Array.prototype.forEach`-invocations with `for..of` instead
This commit is contained in:
commit
d588f5f961
1 changed files with 5 additions and 7 deletions
|
@ -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),
|
||||
});
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue