Skip to content

fix: Correct tooltip field for faceted choropleth example - #3920

Merged
joelostblom merged 3 commits into
vega:mainfrom
Sparlitu:faceted-choropleth-wrong-tooltip-fix
Nov 14, 2025
Merged

joelostblom merged 3 commits into
vega:mainfrom
Sparlitu:faceted-choropleth-wrong-tooltip-fix

Conversation

@Sparlitu

Copy link
Copy Markdown
Contributor

Fix #3919.

This PR aims to provide the required changes to the tooltip reference in order to display the correct County id

@joelostblom joelostblom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @Sparlitu !

@joelostblom joelostblom changed the title fix: Faceted choropleth wrong tooltip reference fix: Correct tooltip field for faceted choropleth example Nov 14, 2025
@joelostblom
joelostblom merged commit ae5579b into vega:main Nov 14, 2025
16 checks passed
@Sparlitu

Copy link
Copy Markdown
Contributor Author

Hi, @joelostblom! I just noticed that my latest commit doesn't contain the necessary modifications for the examples_methods_syntax, only for the arguments one. The correct commit is the aefb97f one. Can we do something about this?

@joelostblom

Copy link
Copy Markdown
Contributor

Ah I didn't notice that. Could you open a new PR from that commit (or a new one with the remaining modification)?

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.

faceted choropleth has wrong tooltip reference: County ID: undefined

2 participants