Skip to content

PB 773 clickable droplet search - #minor #1031

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Aug 27, 2024

Conversation

ismailsunni
Copy link
Contributor

@ismailsunni ismailsunni commented Aug 19, 2024

New screenshot:
image

Test link

@github-actions github-actions bot changed the title PB 773 clickable droplet search PB 773 clickable droplet search - #minor Aug 19, 2024
@ismailsunni ismailsunni requested review from pakb and ltshb August 19, 2024 04:36
Copy link

cypress bot commented Aug 19, 2024

web-mapviewer    Run #3126

Run Properties:  status check passed Passed #3126  •  git commit c482319a90: PB-773: Remove deselect callback since it's not used.
Project web-mapviewer
Branch Review pb-773-clickable-droplet-search
Run status status check passed Passed #3126
Run duration 04m 34s
Commit git commit c482319a90: PB-773: Remove deselect callback since it's not used.
Committer Ismail Sunni
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 210
View all changes introduced in this branch ↗︎

Copy link
Contributor

@ltshb ltshb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to differentiate 2 different use case here:

  1. Search result is a place (coordinate or address)
  2. Search result is an object (enable the RBD: building status layer and type something like bern)

In the first use case I would reuse the location popup as the user might be interested at any coordinate or might be want to share his address/place to someone using for example whatsapp.

In the second use case I see that we already have a different behavior from the old viewer, where on the old viewer it selected the object and showed its content, it did not add a droplet. In that case we should have the same behavior as on the old viewer simply select the object (similar as if the user clicked the object).

Moreover this PR is breaking the infobox behavior to toggle between infobox and popup.

@ismailsunni
Copy link
Contributor Author

So, in short:
First use case: show the location pop up
Second use case: remove the droplet and select the object (then automatically show the info box for the object)

Regarding the toggle behaviour, there is a ticket to remove the behaviour, and use can show both location popup and infobox (PB-859).

@ismailsunni ismailsunni force-pushed the pb-773-clickable-droplet-search branch from e81a553 to 4bab433 Compare August 26, 2024 10:26
@ismailsunni
Copy link
Contributor Author

ismailsunni commented Aug 26, 2024

hi @ltshb
I have updated the approach. Now, when you click the pin, it will show the pop up location (as shown in the screenshot above).

I will implement the second use case in another PR, since the original issue is clicking on the droplet.

Thanks

@ismailsunni ismailsunni requested a review from ltshb August 26, 2024 10:48
@ismailsunni ismailsunni merged commit b118981 into master Aug 27, 2024
9 checks passed
@ismailsunni ismailsunni deleted the pb-773-clickable-droplet-search branch August 27, 2024 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants