Release/10.6.2 #5103
Closed
Release/10.6.2 #5103
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
window.scrollY is already account for which means document.body.scrollTop incorrectly offsets the tooltip vertically. The same is not true for horizontal position.
Position the tooltip centered, just below the node being hovered. Update packages/mermaid/src/diagrams/flowchart/flowDb.js Co-authored-by: Sidharth Vinod <[email protected]>
Originally, the function was checking if any commits were on the same branch as `commit2`, the destination commit. However, in order to avoid a conflict, we should only need to check whether any commits are on the same branch as `commit 1`. Updated and moved commenting as well.
Changed argument names from commit1 and commit2 to commitA and commitB respectively to prevent confusion with seq number values. Replaced Array filter method with array some method so that as soon as one overlap is found, function is finished. Used Object.entries instead of Object.keys to reduce number of variables needed and make function easier to read.
On previous rewrite, I had created new functions within the overlapping functions but these were being recreated on each iteration of Object.some(). I moved them outside this for clarity and so they're not recreated each iteration.
The function also now does an additional check to see if source branch in overlap check is on main. As we're no longer purely checking for an overlap and the only use of this function is to reroute the arrows to avoid running over commits, this more literal name should be clearer.
Checking if branch was same as main turned out to be redundant for now, since there don't seem to be any cases where routing curves into main. This fixes issue found in review by @nirname and avoids a situation where branching from the same commit results in unnecessary rerouting.
Fix for error spotted by @mathbraga
Followed practice of other tests so that commit IDs are stabilised (i.e., not randomly generated) and therefore don't show repeatedly in Cypress diff output screenshots
Hypothesised that working out which branch needed checking for overlapping commits might be missing, so added that as a nested ternary and passed result as new argument to rerouting check. If commits are found on the branch which will be getting the curve (whichever branch is lower or more to the right of main than the other, for now), then the arrow will be rerouted. I may refactor in a follow-up commit and I think there's scope to simplify the logic but this is a test for now.
My focus earlier on had been on relationship to `main` branch so this is to ensure that we have some tests that cover relationship between a pair of branches that doesn't include `main`.
…maid-js/mermaid into 2408-support-style-for-class
Added functionality to support style keyword
Adds Unison programming language to community integrations list
chore(deps): update all minor dependencies (minor)
* develop: (200 commits) chore(deps): update all minor dependencies adds corresponding change in docs/ecosystem Adds Unison programming language to community integrations list Fixed parser/tests Update docs Update classDiagram.md Update classDiagram.md Update docs Update packages/mermaid/src/diagrams/class/classDb.ts Update packages/mermaid/src/docs/syntax/classDiagram.md Update packages/mermaid/src/diagrams/class/classDb.ts chore(deps): update all minor dependencies Update generics docs Update docs Address potential undefined refactor: Move maxEdges out of flowchart config. refactor: Move maxEdges out of flowchart config. chore: Add maxEdges to secure list Update docs Update NiceGuy.io links in integrations-community.md ...
…ceDb-variables fix/1294_exhaustive-clear-sequenceDb-variables
* develop: (22 commits) chore(deps): update all minor dependencies adds corresponding change in docs/ecosystem Adds Unison programming language to community integrations list Fixed parser/tests Update docs Update classDiagram.md Update classDiagram.md Update docs Update packages/mermaid/src/diagrams/class/classDb.ts Update packages/mermaid/src/docs/syntax/classDiagram.md Update packages/mermaid/src/diagrams/class/classDb.ts chore(deps): update all minor dependencies Update generics docs Update docs Address potential undefined refactor: Move maxEdges out of flowchart config. refactor: Move maxEdges out of flowchart config. chore: Add maxEdges to secure list Update packages/mermaid/src/diagrams/class/classDb.ts Update docs ...
* master: add jetbrains blog post additional updates update latest news pages fix addl broken link fix link build: use `tsx` instead of `ts-node-esm` verbiage updates feat: Track outbound links in plausible update getting started page add blog post chore: Update promo link docs: Holiday promo v2 Update packages/mermaid/src/docs/.vitepress/components/TopBar.vue update announcements page update announcement bar verbiage
* 'develop' of github.com:mermaid-js/mermaid: chore: Minor cleanup of imperativeState fix: replace functional approaches with oop chore: fix autogen docs chore(sequence): Update packages/mermaid/src/docs/syntax/sequenceDiagram.md chore(sequence): update doc for actors/participant creation/deletion fix chore: remove unused e2e tests file chore: add e2e test that shows db cleanup problem chore: add e2e test that shows db cleanup problem fix: add imperativeState and replace sequenceDb global variables with it
* develop: (24 commits) chore: Minor cleanup of imperativeState add jetbrains blog post additional updates update latest news pages fix addl broken link fix link build: use `tsx` instead of `ts-node-esm` verbiage updates fix: replace functional approaches with oop feat: Track outbound links in plausible update getting started page add blog post chore: Update promo link docs: Holiday promo v2 Update packages/mermaid/src/docs/.vitepress/components/TopBar.vue update announcements page update announcement bar verbiage chore: fix autogen docs chore(sequence): Update packages/mermaid/src/docs/syntax/sequenceDiagram.md chore(sequence): update doc for actors/participant creation/deletion fix ...
* develop: docs: Remove broken integration
DOCS: update Flowchart page
* develop: update verbiage update verbiage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.