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

Omit root graph name unless explicitly assigned via graphviz.name attribute #28

Merged
merged 2 commits into from
Oct 9, 2018

Conversation

clue
Copy link
Member

@clue clue commented Oct 9, 2018

This library now accepts an optional graphviz.name attribute that will be used as the name (or ID) for the root graph object in the DOT output if given. Unless explicitly assigned, this will now be omitted by default. It is common to assign a G here, but usually there should be no need to assign this. Among others, this may be used as the title or tooltip in SVG output.

Supersedes / closes #20, thank you @rhelms
Resolves / closes #3

@clue clue added this to the v0.3.0 milestone Oct 9, 2018
@clue clue merged commit 80e5b26 into graphp:master Oct 9, 2018
@clue clue deleted the omit-name branch October 9, 2018 13:43
@clue clue mentioned this pull request Mar 9, 2019
@clue clue added BC break and removed BC break labels Oct 4, 2019
@clue clue modified the milestones: v0.3.0, v0.2.2 Oct 4, 2019
@clue clue removed the BC break label Oct 4, 2019
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.

Add an option to define the graph ID in dot script
2 participants