📑 Summary

This PR fixes the wrong behavior of direction TD which should've behaved like TB.

In flowDb.ts, the method setDirection takes dir string argument representing the direction. The data received had an extra space ' ' before the direction identifier, e.g. " TD". There is a check if the direction is TD, convert it to TB with strict equality check.

Resolves #6681

📏 Design Decisions

In flowDb.ts, the method setDirection takes dir string argument representing the direction. The data received had an extra space ' ' before the direction identifier, e.g. " TD". There is a check if the direction is TD, convert it to TB with strict equality check. Thus, " TD" === "TD" is always false, hence not converted to "TB" as shown below.

if (this.direction === 'TD') {
      this.direction = 'TB';
    }

To fix this, trimming the string is the best way instead of changing the right hand side of the equality check or using another condition.

-    this.direction = dir;
+    this.direction = dir.trim();

Test with the following:

flowchart TD
    subgraph one
        a2
        a1
    end
    subgraph two
        b2
        b1
    end
    a1 --> a2
    b1 --> b2
    one --> two

📋 Tasks

Make sure you

• 📖 have read the contribution guidelines
• 💻 have added necessary unit/e2e tests.
• 📓 have added documentation. Make sure MERMAID_RELEASE_VERSION is used for all new features.
• 🦋 If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.