Skip to content
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

Filtering being deactivated after clicking away from metadata popup #1701

Closed
kvargha opened this issue Sep 20, 2023 · 6 comments · Fixed by #1749
Closed

Filtering being deactivated after clicking away from metadata popup #1701

kvargha opened this issue Sep 20, 2023 · 6 comments · Fixed by #1749
Labels
bug Something isn't working

Comments

@kvargha
Copy link

kvargha commented Sep 20, 2023

Current Behavior

When you click on a filtered sample and click away from the popup, the filter becomes deactivated.

Expected behavior

The filtered shouldn't be deactivated.

How to reproduce

Steps to reproduce the current behavior:

  1. Visit https://nextstrain.org/fetch/storage.googleapis.com/ucsc-gi-cdph-bigtree/CDPH_09_20_2023/CDPH-subtree-178.json?s=USA/CA-CDPH-2000016698/2021%7CEPI_ISL_2782613%7C2021-01-04,USA/CA-CDPH-2000016731/2021%7CEPI_ISL_2782386%7C2021-01-03
  2. Click "Zoom To Selected"
  3. Click on a sample
  4. Click away the popup.
  5. Samples filter should be deactivated.

Your environment: if browsing Nextstrain online

  • Operating system: macOS 13.5.2 (22G91)
  • Browser: Chrome Version 117.0.5938.88

Additional context

Auspice Recording

@sabbymama
Copy link

I'm being ghosted by my sequences whenever I click on them. Will this be fixed soon?

@joverlee521
Copy link
Contributor

Noting this bug is part of the larger issue of the tip modal state not being accurately tracked in the redux state as described by @jameshadfield in #1243 (comment).

@kvargha
Copy link
Author

kvargha commented Jan 31, 2024

Thank you for providing the background @joverlee521. Is there an estimate of when it will be fixed?

jameshadfield added a commit that referenced this issue Feb 5, 2024
Clicking on a tip activates a modal and activates the corresponding
strain filter. Previously clearing the modal would always inactivate the
filter. In the absence of any filtering this behaviour is not bad, but
when already filtering to a set of strains then it's counter-intuitive
because we inactivate a filter that was active.

The new behaviour restores the filtering state of the strain before the
modal was opened. Specifically,
* If the tip was already filtered, opening and closing the modal doesn't
  change the filters.
* If the tip was filtered but inactive, opening the modal activates it
  and closing the modal returns it to the inactive state.
* If the tip wasn't filtered (active or inactive) then opening the modal
  makes it an active filter and closing the modal removes the filter
  entirely.

Closes #1701
jameshadfield added a commit that referenced this issue Feb 6, 2024
Clicking on a tip activates a modal and activates the corresponding
strain filter. Previously clearing the modal would always inactivate the
filter. In the absence of any filtering this behaviour is not bad, but
when already filtering to a set of strains then it's counter-intuitive
because we inactivate a filter that was active.

The new behaviour restores the filtering state of the strain before the
modal was opened. Specifically,
* If the tip was already filtered, opening and closing the modal doesn't
  change the filters.
* If the tip was filtered but inactive, opening the modal activates it
  and closing the modal returns it to the inactive state.
* If the tip wasn't filtered (active or inactive) then opening the modal
  makes it an active filter and closing the modal removes the filter
  entirely.

Closes #1701
@kvargha
Copy link
Author

kvargha commented Feb 12, 2024

@jameshadfield I'm still experiencing this issue.

@victorlin
Copy link
Member

@kvargha the fix was merged and released in Auspice version 2.52.0, but nextstrain.org has not been updated yet. We may do that sometime this week. In the meantime, you can use https://next.nextstrain.org which is automatically updated with the latest version of Auspice.

@kvargha
Copy link
Author

kvargha commented Feb 13, 2024

Gotcha. Thank you @victorlin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants