Skip to content

fix: correctly ignore activity and task label bounds#3434

Draft
tbouffard wants to merge 13 commits intomasterfrom
fix/correctly_ignore_activity_label_bounds
Draft

fix: correctly ignore activity and task label bounds#3434
tbouffard wants to merge 13 commits intomasterfrom
fix/correctly_ignore_activity_label_bounds

Conversation

@tbouffard
Copy link
Member

@tbouffard tbouffard commented Dec 4, 2025

The positions (x and y coordinates) were not ignored.
Only the global positioning of the label, relative to the shape, was updated.

Also rename some visual tests prior adding new ones, including test for ignore label styles as well.

The position (x and y coordinates) were not ignored.
Only the global positioning of the label, relative to the shape was updated.

Also rename some visual tests prior adding new ones.
@tbouffard tbouffard added the bug Something isn't working label Dec 4, 2025
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🎊 PR Preview 171c6b6 has been successfully built and deployed to https://process-analytics-bpmn-visualization-js-demo_preview-pr-3434.surge.sh

🕐 Build time: 0.011s

🤖 By surge-preview

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🎊 PR Preview 171c6b6 has been successfully built and deployed to https://process-analytics-bpmn-visualization-js-doc_preview-pr-3434.surge.sh

🕐 Build time: 0.011s

🤖 By surge-preview

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2025

@tbouffard tbouffard marked this pull request as draft December 5, 2025 09:17
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant