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
docs: update to the zoomable circle packing example #3857
Conversation
… double quote (") from description; added a hyperlink to a more complex example; adjusted width and height of example to match the regular circle packing example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one comment.
@@ -1,8 +1,8 @@ | |||
{ | |||
"$schema": "https://vega.github.io/schema/vega/v5.json", | |||
"description": "An example of a zoomable circle packing layout for hierarchical data.", | |||
"width": 800, | |||
"height": 800, | |||
"width": 600, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to rebuild the image?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure I follow. I'm assuming that you're not referring to the thumbnail image, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the only spec update being the width and height (same aspect ratio), the existing thumbnail looks good in my dev environment and should do the trick. I'm happy to update it though if you think that I should. Just let me know 👍
Removed trailing double quote (") from description; added a hyperlink to a more complex example; adjusted width and height of example to match the regular circle packing example
Note - accidentally overwrote the commits in my local repository for #3856. That PR has been closed. This PR is intended to replace that one.
INFO: @domoritz