-
Notifications
You must be signed in to change notification settings - Fork 326
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
New visualization UI API #11077
New visualization UI API #11077
Conversation
@Frizi The other side of viz API improvement is the static-API change we talked about, but I think that change will mostly affect different code from this, so let's merge this separately since it contains API changes and several bugfixes. |
<span v-text="props.data.error.message"></span> | ||
</div> | ||
<div class="LoadingErrorVisualization"> | ||
<div> |
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.
Probably don't need the inner div anymore.
Pull Request Description
size
property reflects the current renderable area.VisualizationContainer
; the visualization simply renders its content at its root.Other API changes:
Interaction
no longer includesGraphNavigator
with pointer events.Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.