-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Documentation Issue, Dagre positioning unclear! #368
Comments
Yep keeping these two consistent is definitely better, thanks for the heads up! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the Bug
There are two examples of how to layout something, both with sample code:
However, option number 1, has a critical difference from option number two, it is missing the following:
This is important because if you copy and past the code from option number one, you probably won't realize there is an issue until much later on.
I would suggest
Thank you for such an awesome library!
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Go to the docs and see that the example for here https://reactflow.dev/learn/layouting/layouting#dagre has a bug that is addressed here https://reactflow.dev/examples/layout/dagre
Expected behavior
the code example should have the fix:
Screenshots or Videos
No response
Platform
NA
Additional context
Thank you!
The text was updated successfully, but these errors were encountered: