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

Pin map with null locations shows tooltip of the wrong row #18063

Closed
flamber opened this issue Sep 26, 2021 · 1 comment · Fixed by #22425
Closed

Pin map with null locations shows tooltip of the wrong row #18063

flamber opened this issue Sep 26, 2021 · 1 comment · Fixed by #22425
Assignees
Labels
.Correctness .Frontend Priority:P2 Average run of the mill bug .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Maps
Milestone

Comments

@flamber
Copy link
Contributor

flamber commented Sep 26, 2021

Describe the bug
Tooltips of Pin maps with null locations shows information from the wrong results.

Workaround Since Metabase will not show pins for null locations, then add a filter to not include those rows in results.

To Reproduce

  1. Native query > Sample Dataset
select null "LATITUDE", null "LONGITUDE", null "COUNT", 'NULL ROW' "NAME"
union all select 55.6761, 12.5683, 1, 'Copenhagen'
  1. Change visualization to Pin map and hover the pin, which shows tooltip information from another row
    The example is simplified, it gets even worse, when there's a mix of null rows in the results
    image

Information about your Metabase Installation:
Tested 0.37.8 thru 0.40.5 and master 67470a0

Additional context
Perhaps related to #12674.

@nemanjaglumac

This comment has been minimized.

nemanjaglumac added a commit that referenced this issue Sep 27, 2021
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label Sep 27, 2021
@flamber flamber added this to the 0.44 milestone May 6, 2022
This was referenced Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Correctness .Frontend Priority:P2 Average run of the mill bug .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Maps
Projects
Development

Successfully merging a pull request may close this issue.

3 participants