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

Add ability to download a query plan as png #624

Merged
merged 5 commits into from
Aug 18, 2017

Conversation

pe4cey
Copy link
Contributor

@pe4cey pe4cey commented Aug 9, 2017

In the cypher result frame for EXPLAN||PROFILE query we display the plan view that has an interactive query plan element.

This PR adds a button in the titlebar to download the query plan as a png file.

@@ -49,6 +49,8 @@ export class PlanView extends Component {
this.ensureToggleExpand(props)
}
shouldComponentUpdate (props, state) {
this.props.assignVisElement && this.props.assignVisElement(this.el, this.plan)
Copy link
Member

@oskarhane oskarhane Aug 10, 2017

Choose a reason for hiding this comment

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

I don't think this is the lifecycle method to use for this. componentWillUpdate or componentWillReceiveProps would be more appropriate imo.

@pe4cey
Copy link
Contributor Author

pe4cey commented Aug 11, 2017

Updated

@oskarhane
Copy link
Member

If you generate multiple frames with plans in them, export as PNG will download the latest plan, even from older frames.

@oskarhane oskarhane force-pushed the 3.0-download-plan-png branch from 6373895 to da3954e Compare August 18, 2017 14:24
@oskarhane oskarhane merged commit 970f4fa into neo4j:3.0 Aug 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants