Modify the Get Started vignette with more appropriate data#671
Merged
JohnCoene merged 4 commits intoJohnCoene:masterfrom Oct 20, 2025
Merged
Modify the Get Started vignette with more appropriate data#671JohnCoene merged 4 commits intoJohnCoene:masterfrom
JohnCoene merged 4 commits intoJohnCoene:masterfrom
Conversation
Accent that the charts can be interacted with
Owner
|
Thank you! This is indeed much better than what we had originally. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The main change is using the
airqualitydataset so that we can build a line chart that has a sensible X axis. Original charts have lines rising and dropping between different US states. Since there is nothing linking data from one state to another, it's not really good practice to make charts that insinuate that population or income rises and falls as you move along the X axis.airqualityhas date information which is more relevant for line charts.Additionaly, some text sections are updated. For example, a sentence has been added to indicate that the charts are interactable by default, prompting users to turn off one of the traces and see that the chart automatically updates the y-axis so that the remaining data can be seen in more detail.
In the Navigate options section an additional argument has been added. The
backgroundColorfor the tooltip has been changed to white to show that not all arguments need to be wrapped in a list. And it also makes the tooltip more readable :)Fixed minor typos.