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

[DOC] Choose a convenient id for html elements displaying the BPMN diagrams #866

Closed
5 tasks done
tbouffard opened this issue Nov 6, 2020 · 1 comment
Closed
5 tasks done
Assignees
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@tbouffard
Copy link
Member

tbouffard commented Nov 6, 2020

Current status

The goal of this decision is to be consistent in both documentation, code and examples.
It is part of the work engaged with #864

Over the time, we have introduced various ids and none is satisfying

  • graph and maybe graph-container: this is the id used in mxgraph, but we aren't a mxgraph lib, we should find something related to the BPMN domain (we are not displaying graphs)
  • viewport, view-port or bpmn-viewport: viewport is a term already used in the browser glossary and may be confusing

Note: other libs use

  • bpmn-js: canvas
  • ProcessMaker/modeler: paper-container

Proposals

  • bpmn-container
  • bpmn-viewport
  • ....

Tasks

@tbouffard
Copy link
Member Author

Closing as all tasks are completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants