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

[FEAT] Update the monitoring demo to use the new Style API #463

Closed
2 tasks done
tbouffard opened this issue Feb 21, 2023 · 0 comments · Fixed by #486
Closed
2 tasks done

[FEAT] Update the monitoring demo to use the new Style API #463

tbouffard opened this issue Feb 21, 2023 · 0 comments · Fixed by #486
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@tbouffard
Copy link
Member

tbouffard commented Feb 21, 2023

Is your feature request related to a problem? Please describe.
The monitoring demo style edges by adding CSS classes. When the zoom level change, the stroke width doesn't follow the zoom level.
So, the stroke width may appear too large or too small depending on the zoom level.
This is the same for the arrow size.

Frequency use case paths not clearly

zoom level 1 zoom level 2
image image

Time use case paths not clearly

zoom level 1 zoom level 2
image image

Describe the solution you'd like
process-analytics/bpmn-visualization-js#1030 will provide a new way to style the BPMN Elements. Comparing to the CSS way, using the API set the value in the SVG DOM elements. So, the size of the stroke won't depend on the zoom level and then, the relative

Describe alternatives you've considered
Keep CSS classnames and still have the limitation.

Additional context
Once implemented, the image of the Process Analytics website should be updated as it displays a screenshot of this demo. Do something similar to process-analytics/process-analytics.dev#899.

image

Tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
2 participants