Skip to content

Commit 2ef890a

Browse files
committed
Fix flashing suggestion in digitransit-component-autosuggest.
1 parent ccd3588 commit 2ef890a

4 files changed

Lines changed: 11 additions & 5 deletions

File tree

digitransit-component/packages/digitransit-component-autosuggest-panel/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
"author": "Digitransit Authors",
2929
"license": "(AGPL-3.0 OR EUPL-1.2)",
3030
"peerDependencies": {
31-
"@digitransit-component/digitransit-component-autosuggest": "^7.1.10",
31+
"@digitransit-component/digitransit-component-autosuggest": "^7.1.11",
3232
"@digitransit-component/digitransit-component-icon": "^2.0.2",
3333
"@hsl-fi/sass": "1.0.0",
3434
"classnames": "2.5.1",

digitransit-component/packages/digitransit-component-autosuggest/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@digitransit-component/digitransit-component-autosuggest",
3-
"version": "7.1.10",
3+
"version": "7.1.11",
44
"description": "digitransit-component autosuggest module",
55
"main": "index.js",
66
"files": [

digitransit-component/packages/digitransit-component-autosuggest/src/index.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -522,9 +522,15 @@ function DTAutosuggest({
522522
openMenu();
523523
};
524524

525-
// Fetch suggestions when isOpen, value, or fetchSuggestions dependies change
525+
// Fetch suggestions when isOpen, value, or fetchSuggestions dependencies change
526526
useEffect(() => {
527-
if (isOpen || state.renderMobile) {
527+
// Don't search when the search field (state.value) contains position strings that were given as a prop (value),
528+
// because they are UI labels not real search terms. Fetching suggestions with them caused a quickly flashing set of
529+
// irrelevant results, e.g. 'Käytä nykyistä sijaintia' -> 'City-käytävä'.
530+
if (
531+
(isOpen || state.renderMobile) &&
532+
!(state.value === value && positions.includes(value))
533+
) {
528534
fetchSuggestions(state.value);
529535
}
530536
}, [isOpen, state.renderMobile, state.value, fetchSuggestions]);

digitransit-component/packages/digitransit-component/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"url": "git://github.com/HSLdevcom/digitransit-ui.git"
1919
},
2020
"dependencies": {
21-
"@digitransit-component/digitransit-component-autosuggest": "^7.1.10",
21+
"@digitransit-component/digitransit-component-autosuggest": "^7.1.11",
2222
"@digitransit-component/digitransit-component-autosuggest-panel": "^8.3.5",
2323
"@digitransit-component/digitransit-component-control-panel": "^7.1.3",
2424
"@digitransit-component/digitransit-component-favourite-bar": "^5.0.5",

0 commit comments

Comments
 (0)