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] Create a demo for all navigation and scrolling features #170

Open
Tracked by #171
tbouffard opened this issue Nov 14, 2023 · 0 comments
Open
Tracked by #171

[FEAT] Create a demo for all navigation and scrolling features #170

tbouffard opened this issue Nov 14, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@tbouffard
Copy link
Member

tbouffard commented Nov 14, 2023

Is your feature request related to a problem? Please describe.

We have several new features that we want to implement, so we will need a demo to demonstrate them

Describe the solution you'd like

Provide a demo with a large diagram that cannot be fully displayed in the BPMN container without doing fit on load. This will make all navigation and scroll features a must have for the demo.
The zoom reset should be a Fit.None to ensure that the diagram is never fully displayed.

The minimap could be displayed with a transparent background as in process-analytics/bpmn-visualization-js#555
The demo should allow to use a BPMN Container with or without scrollbars

demo mockup bpmn-visualization BPMN Navigation new features
Mockup done with https://moqups.com

Additional context

Be aware of #80.

Sharing "zoom controls"

We already have a "zoom controls" button group in the "OverlayPlugin demo".
We could/should avoid duplication and create a component or function to share the code.
We could introduce a JS function that add elements in the DOM

  • put title on all buttons
  • instead of fit, add an icon and the title should be "Reset zoom level"

[UPDATE] A dedicated component was introduced in #186

Existing zoom bugs

Be aware of this bug about the zoom mouse support with scrollbars: process-analytics/bpmn-visualization-js#1979

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
None yet
Development

No branches or pull requests

1 participant