From 9ee83362f020fb50594a2d50ef0d7ea4c6879bf5 Mon Sep 17 00:00:00 2001 From: Kristofer Baxter Date: Tue, 27 Feb 2018 18:06:13 -0800 Subject: [PATCH] Move away from querySelector and querySelectorAll --- src/shared/dom.js | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/shared/dom.js b/src/shared/dom.js index 9003faee3e..7b2586f855 100644 --- a/src/shared/dom.js +++ b/src/shared/dom.js @@ -169,12 +169,15 @@ export function selectOptions(select, value) { } export function selectValue(select) { - var selectedOption = select.querySelector(':checked') || select.options[0]; - return selectedOption && selectedOption.__value; +  var selectedOption = select.options[select.selectedIndex >= 0 ? select.selectedIndex : 0]; +  return selectedOption && selectedOption.__value; } export function selectMultipleValue(select) { - return [].map.call(select.querySelectorAll(':checked'), function(option) { - return option.__value; - }); +  var values = []; + for (var i = 0; i < select.options.length; i += 1) { + var option = select.options[i]; +    option.selected && values.push(option.__value); +  } +  return values; } \ No newline at end of file