Skip to content

PB-1246 : add location search entry title to search bar when selected#1163

Merged
pakb merged 1 commit intodevelopfrom
bug-PB-1246-location-search
Dec 9, 2024
Merged

PB-1246 : add location search entry title to search bar when selected#1163
pakb merged 1 commit intodevelopfrom
bug-PB-1246-location-search

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Dec 9, 2024

users were confused that the full address of what they selected in the search result wasn't properly shown in the UI.

This only adds the title of such entries to the search bar, without triggering a new search. This means the search results aren't cleared/emptied when a location is selected.

Test link

users were confused that the full address of what they selected in the search result wasn't properly shown in the UI.

This only adds the title of such entries to the search bar, without triggering a new search. This means the search results aren't cleared/emptied when a location is selected.
@github-actions github-actions bot added the bug label Dec 9, 2024
@cypress
Copy link

cypress bot commented Dec 9, 2024

web-mapviewer    Run #4039

Run Properties:  status check passed Passed #4039  •  git commit cb1884b60b: PB-1246 : add location search entry title to search bar when selected
Project web-mapviewer
Branch Review bug-PB-1246-location-search
Run status status check passed Passed #4039
Run duration 05m 10s
Commit git commit cb1884b60b: PB-1246 : add location search entry title to search bar when selected
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 215
View all changes introduced in this branch ↗︎

@pakb pakb merged commit 403439d into develop Dec 9, 2024
@pakb pakb deleted the bug-PB-1246-location-search branch December 9, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants