This repository has been archived by the owner on Jul 29, 2019. It is now read-only.
Add parsing penwidth attribute #3891
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As described in TODO, dotparser.js does not support 'arrowhead' attribubte of edge. This update is for adding 'dot' and 'tee'(bar) styles.
To support box arrowhead of graphviz, add Box endpoint class in EndPoints.js and box attribute in dotparser.js.
To support diamond arrowhead of graphviz, add Diamond endpoint class in EndPoints.js and diamond attribute in dotparser.js.
To support crow arrowhead of graphviz, add Crow endpoint class in EndPoints.js and crow attribute in dotparser.js.
To support normal arrowhead of graphviz, add Triangle endpoint class in EndPoints.js and normal attribute in dotparser.js.
To support curve arrowhead of graphviz, add Curve endpoint class in EndPoints.js and curve attribute in dotparser.js.
To support inverted curve arrowhead of graphviz, add InvertedCurve endpoint class in EndPoints.js and icurve attribute in dotparser.js.
To support vee arrowhead of graphviz, add Vee endpoint class in EndPoints.js and vee attribute in dotparser.js.
In createEdge(), accessing 'attr' causes an error if the edge has no attribute and the value is null. This update fixes bug for accessing null 'attr'.
Add followingn options for 'arrows.to.type'. * box * crow * curve * diamond * inv_curve * triangle * inv_triangle * vee
Add arrow types for the example. 'box', 'crow', 'curve', 'inv_curve', 'diamond', 'triangle', 'inv_triangle', 'vee'
Directional arrow has 'dir' attribute for defining direction of 'forward', 'back', 'both' or 'none'. This update enables to 'dir' attribute. If you define an edge as 'a -> b[arrowhead=normal, dir=both]', arrowheads are depicted both of source and destination.
This update is for comply with graphviz more strictly. As explained in [1], graphviz has limitations for combination of arrow[head|tail] and dir but dotparser.js does not behaviour as same. [1] https://www.graphviz.org/doc/info/attrs.html#h:undir_note If attribute list includes 'dir', following cases just be supported. 1. both or none + arrowhead, arrowtail 2. forward + arrowhead (arrowtail is not affedted) 3. back + arrowtail (arrowhead is not affected) In this update, parsed attributes is pushed 'attr_list' at once and setValue() is not called in while loop in parseAttributeList(). setValue() is called after while loop ends. It is because for checking attributes which change the same attribute of vis. For instance, both of 'arrowhead' and 'dir' (edge style defined in DOT) make changes for the same 'arrows' attribute in vis. If setValue() is called from while loop, previous change is overwritten.
To reduce the number of statements of parseAttributeList(), move parsing 'dir' attribute to function parseDirAttribute() because it is going to reach max-statements limitation of lint.
'penwidth' defines the line width as 'width'. If both are exist, 'penwidth' is used and 'width' is discarded as described in [1]. [1] https://www.graphviz.org/doc/info/attrs.html#d:penwidth
Merge remote-tracking branch 'upstream/develop' into add_penwidth
|
@geminoa, also, could you include an example that uses the new support for the |
|
@micahstubbs , OK, I'll update it. |
|
@geminoa this might be helpful - here is a view that compares the |
17 tasks
29 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This update is for adding 'penwidth' attribute which defines the line width as 'width'. If both are exist, 'penwidth' is used and 'width' is discarded as described in [1].
[1] https://www.graphviz.org/doc/info/attrs.html#d:penwidth