Skip to content

refactor: simplify GraphConfigurator#3143

Merged
tbouffard merged 2 commits into
masterfrom
refactor/simplify_GraphConfigurator
Aug 21, 2024
Merged

refactor: simplify GraphConfigurator#3143
tbouffard merged 2 commits into
masterfrom
refactor/simplify_GraphConfigurator

Conversation

@tbouffard

Copy link
Copy Markdown
Member

Remove the container property as it is not used (it was defined in the constructor)
Panning cursor: remove extra condition as the graph is always enabled

Remove the container property as it is not used (it was defined in the constructor)
Panning cursor: remove extra condition as the graph is always enabled
@tbouffard tbouffard added the refactoring Code refactoring label Aug 14, 2024
@github-actions

github-actions Bot commented Aug 14, 2024

Copy link
Copy Markdown

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

🕐 Build time: 0.01s

🤖 By surge-preview

@github-actions

github-actions Bot commented Aug 14, 2024

Copy link
Copy Markdown

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

🕐 Build time: 0.012s

🤖 By surge-preview

@tbouffard tbouffard marked this pull request as ready for review August 14, 2024 14:45
@tbouffard tbouffard requested a review from csouchet August 14, 2024 14:45
@sonarqubecloud

Copy link
Copy Markdown

@tbouffard tbouffard merged commit 4328c44 into master Aug 21, 2024
@tbouffard tbouffard deleted the refactor/simplify_GraphConfigurator branch August 21, 2024 09:11
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

Successfully merging this pull request may close these issues.

2 participants