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

Subgraph direction not always respected #3096

Closed
SandroK opened this issue Jun 2, 2022 · 3 comments
Closed

Subgraph direction not always respected #3096

SandroK opened this issue Jun 2, 2022 · 3 comments
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect

Comments

@SandroK
Copy link

SandroK commented Jun 2, 2022

Describe the bug
The direction statement of a subgraph is not respected, when pointing at a node within the subgraph

To Reproduce
Compare the following charts, the subgraph direction is different, but shouldn't be

Pointing at the subchart as a whole, subchart direction is top to bottom

flowchart LR;
	subgraph c
		direction TB
		c1 --> c2
	end
	b --> c

Pointing at a node within the subchart, subchart direction is left to right (as the entire chart)

flowchart LR;
	subgraph c
		direction TB
		c1 --> c2
	end
	b --> c1

Expected behavior
Subgraph direction should be respected, independent of other nodes and connections

System info
I have tried on mermaid.live
I believe my system info sn't relevant

@SandroK SandroK added Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect labels Jun 2, 2022
@Zingam
Copy link

Zingam commented Jun 5, 2022

flowchart LR
  start(START) --> user
  user(User UI) --> R_Server(Server) 

  subgraph R Server
    direction TB

    R_Server --> RMCE_Database[(Database)]
  end

  R_Server -->|Request| R_Agent(Agent)

  subgraph R Agent
    direction TB

    R_Agent --> DataCollect(((Data Collection)))
  end

  R_Agent -->|Response| R_Server
Loading

Not respected: direction TB for subgraphs.

Some hints about positioning the elements would be nice too.

@taranlu-houzz
Copy link

@SandroK I think this is a duplicate of #2509. I just ran into this recently as well.

@SandroK
Copy link
Author

SandroK commented Jun 27, 2022

I'll accept that as a duplicate

@SandroK SandroK closed this as completed Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Triage Needs to be verified, categorized, etc Type: Bug / Error Something isn't working or is incorrect
Projects
None yet
Development

No branches or pull requests

3 participants