-
Notifications
You must be signed in to change notification settings - Fork 227
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 optional tweak per node with name placeholder #357
base: dev
Are you sure you want to change the base?
Conversation
See my newly extended #270 (comment) for a use case with example YAML input for this PR. |
I also realized that the templates and examples do not contain any tweaks. The automated checks would only confirm that this PR does not break any of them. |
Implements feature suggested in #349
I wrote:
@kvid : Taking the latter thought a bit further, I come to think of it as regression testing for solved issues... Quite a bit of work to implement though. |
@martinrieder wrote:
True
It would perhaps be nice with an example, but bear in mind that tweaks are mainly needed for workarounds when the ordinary functionality is not sufficient or contains errors, or to test possible new features. Hopefully, such needs will be resolved by extending the ordinary functionality in the future.
Creating tweaks does require knowledge of the Graphviz syntax, and that is a too large subject to cover in the WireViz documentation, so referring to Graphviz docs is needed.
#251 will add test folders that might be what you seek.
See #63 |
I would put some notes into syntax.md and amend those to my PR #386 then. We have seen a few requests about clustering. That is something that I would still like to add, besides the hint about the image attributes. |
Implements feature suggested in #349