Skip to content

Commit ca44599

Browse files
authored
feat: add support for query suggestions on select options (#252)
1 parent 935f671 commit ca44599

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

devtools/src/content-script/highlighter/index.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ export default function setupHighlighter({
6565
view.addEventListener('pointerdown', onPointerDown, true);
6666
view.addEventListener('pointerover', onPointerOver, true);
6767
view.addEventListener('pointerup', onPointerUp, true);
68+
view.addEventListener('change', onChange, true);
6869
}
6970
}
7071

@@ -99,6 +100,7 @@ export default function setupHighlighter({
99100
view.removeEventListener('pointerdown', onPointerDown, true);
100101
view.removeEventListener('pointerover', onPointerOver, true);
101102
view.removeEventListener('pointerup', onPointerUp, true);
103+
view.removeEventListener('change', onChange, true);
102104
}
103105
}
104106

@@ -164,6 +166,16 @@ export default function setupHighlighter({
164166
stopPropagation(event);
165167
}
166168

169+
function onChange(event) {
170+
stopPropagation(event);
171+
172+
if (event.target.tagName.toLowerCase() === 'select') {
173+
const select = event.target;
174+
const option = select.options[select.selectedIndex];
175+
selectNode(option, { origin: 'click' });
176+
}
177+
}
178+
167179
const selectNode = throttle(
168180
memoize(onSelectNode),
169181
200,

src/lib/queryAdvise.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ export function getData({ rootNode, element }) {
2727
labelText: labelText,
2828
placeholderText: element.getAttribute('placeholder'),
2929
text: element.innerText,
30-
displayValue: element.getAttribute('value'),
30+
displayValue:
31+
tagName.toLowerCase() === 'option' ? '' : element.getAttribute('value'),
3132

3233
altText: element.getAttribute('alt'),
3334
title: element.getAttribute('title'),

0 commit comments

Comments
 (0)