Skip to content

Commit 950ab18

Browse files
committed
refactor(forms): correct Domino reference in selectedOptions fallback comment
Angular uses Domino for server-side rendering, not jsdom. The fallback comment now correctly names Domino as the environment where `selectedOptions` is unavailable.
1 parent fddfa29 commit 950ab18

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

‎packages/forms/signals/src/directive/control_select_multiple.ts‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,8 @@ function getSelectMultipleControlValue(select: HTMLSelectElement): string[] {
7676
return selected;
7777
}
7878

79-
// `selectedOptions` is not available in some environments (e.g. older jsdom versions used
80-
// in Node.js tests). Fall back to iterating the full `options` list and filtering by the
81-
// `selected` flag.
79+
// `selectedOptions` is not implemented in Domino (used for server-side rendering), so fall
80+
// back to iterating the full `options` list and filtering by the `selected` flag.
8281
for (let i = 0; i < select.options.length; i++) {
8382
const option = select.options[i];
8483
if (option.selected) {

0 commit comments

Comments
 (0)