Skip to content

docs: explain mxGraph perimeter and projection#2096

Merged
tbouffard merged 15 commits into
masterfrom
doc/explain_mxgraph_edge_terminal_points_and_perimeter
Feb 4, 2025
Merged

docs: explain mxGraph perimeter and projection#2096
tbouffard merged 15 commits into
masterfrom
doc/explain_mxgraph_edge_terminal_points_and_perimeter

Conversation

@tbouffard

@tbouffard tbouffard commented Jul 6, 2022

Copy link
Copy Markdown
Member

It references the maxGraph documentation and explains how we configure and use perimeters in bpmn-visualization.

Notes

It includes information gathered during development of #1868 and that will be useful for #1870.

Remaining Tasks (more may be needed)

@tbouffard tbouffard added documentation Improvements or additions to documentation WIP 🚧 Pull request in progress and/or not ready for review. Used in addition to marking it as draft labels Jul 6, 2022
@tbouffard tbouffard changed the title [DOC] Explain mxGraph perimeter and projection docs: explain mxGraph perimeter and projection Jun 7, 2024
@tbouffard

Copy link
Copy Markdown
Member Author

ℹ️ I plan to work on this Pull Request again soon.
In the meantime, please don't close this Pull Request.

@tbouffard tbouffard force-pushed the doc/explain_mxgraph_edge_terminal_points_and_perimeter branch from fa11ccb to b0b3326 Compare January 30, 2025 13:12
@tbouffard tbouffard force-pushed the doc/explain_mxgraph_edge_terminal_points_and_perimeter branch from eb22c96 to 069a457 Compare January 31, 2025 16:15
@tbouffard tbouffard removed the WIP 🚧 Pull request in progress and/or not ready for review. Used in addition to marking it as draft label Feb 3, 2025
@tbouffard tbouffard requested a review from csouchet February 3, 2025 12:57
@tbouffard tbouffard marked this pull request as ready for review February 3, 2025 12:57

@csouchet csouchet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great documentation job ✨

I have some suggestions for making the documentation lighter and more pleasant to read.

Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
Comment thread docs/contributors/mxgraph-integration.md Outdated
@tbouffard

Copy link
Copy Markdown
Member Author

@csouchet thanks for the review.
I've integrated your suggestions directly or adapted them before integration.

@tbouffard tbouffard merged commit 873643d into master Feb 4, 2025
@tbouffard tbouffard deleted the doc/explain_mxgraph_edge_terminal_points_and_perimeter branch February 4, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants