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

[REFACTOR] Use the updated API to remove all overlays at once #32

Open
tbouffard opened this issue Mar 13, 2023 · 0 comments
Open

[REFACTOR] Use the updated API to remove all overlays at once #32

tbouffard opened this issue Mar 13, 2023 · 0 comments
Labels
refactoring Code refactoring

Comments

@tbouffard
Copy link
Member

Use process-analytics/bpmn-visualization-js#2577 when available to avoid having to keep all elements that have overlays.

@tbouffard tbouffard added the refactoring Code refactoring label Mar 13, 2023
assynour added a commit that referenced this issue Apr 17, 2023
covers #33

Only overlays are added to edges. Their style is not updated. This will
be addressed in a separate PR when the code is simplified with #3

<img width="699" alt="bpmn-with-edge-stat"
src="https://user-images.githubusercontent.com/67027776/232003024-d96b27ba-e690-4828-808f-74ef22f2fdda.png">

⚠️ Removing overlays is not working properly. It will be fixed with #32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Code refactoring
Projects
None yet
Development

No branches or pull requests

1 participant