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] add various documentation improvements #680

Merged
merged 8 commits into from
Oct 1, 2020
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ If you need BPMN examples, you can use resources from
| Yes | Yes | Yes | Yes |
tbouffard marked this conversation as resolved.
Show resolved Hide resolved

**Note**: Internet Explorer will never be supported. \
The library may work with the other browsers, currently not list, if they support ES6.
The library may work with the other browsers, they must at least support ES6.
tbouffard marked this conversation as resolved.
Show resolved Hide resolved

## 🎨 Features

Expand Down
7 changes: 7 additions & 0 deletions docs/architecture/development.adoc
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
=== Development
:icons: font
ifdef::env-github[]
:tip-caption: :bulb:
:note-caption: :information_source:
:important-caption: :heavy_exclamation_mark:
:caution-caption: :fire:
:warning-caption: :warning:
endif::[]

[TIP]
====
Expand Down
29 changes: 26 additions & 3 deletions docs/bpmn-support.adoc
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
[[supported-bpmn-elements]]

== Supported BPMN Elements
:icons: font
ifdef::env-github[]
:tip-caption: :bulb:
:note-caption: :information_source:
:important-caption: :heavy_exclamation_mark:
:caution-caption: :fire:
:warning-caption: :warning:
endif::[]

NOTE: The BPMN support roadmap is available in https://github.com/process-analytics/bpmn-visualization-js/milestones[Github milestones]

Expand Down Expand Up @@ -163,20 +169,36 @@ The event definition can be defined on the event or on the definitions.
|The stroke & icon width may be adjusted
|===


[cols="1,1,4", options="header"]
.Event Sub-Process Interrupting Start Events
|===
|Name
|Rendering
|Comments

|Timer Interrupting Start Event
|icon:check-circle-o[]
|The icon width may be adjusted

|Message Interrupting Start Event
|icon:check-circle-o[]
|The stroke & icon width may be adjusted

|Error Interrupting Start Event
|icon:check-circle-o[]
|The stroke & icon width may be adjusted

|Signal Interrupting Start Event
|icon:check-circle-o[]
|The stroke & icon width may be adjusted

|Compensation Interrupting Start Event
|
|
|===



[cols="1,1,4", options="header"]
.Event Sub-Process Non-Interrupting Start Events
|===
Expand All @@ -188,7 +210,7 @@ The event definition can be defined on the event or on the definitions.
|icon:check-circle-o[]
|The icon width may be adjusted

|MessageNon-interrupting Start Event
|Message Non-interrupting Start Event
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

|icon:check-circle-o[]
|The stroke & icon width may be adjusted

Expand All @@ -197,6 +219,7 @@ The event definition can be defined on the event or on the definitions.
|The stroke & icon width may be adjusted
|===


[cols="1,1,4", options="header"]
.Intermediate Catch Events
|===
Expand Down
13 changes: 12 additions & 1 deletion docs/development/bpmn-support-how-to.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ flow` support (see `ShapeBpmnElementKind` for more details)
Overview of tasks to be achieved:
- use the final icon chosen for the BPMN Elements.
- add/update visual tests
- update the BPMN support documentation
- update the BPMN support documentation (see also [icons license](#icons-license))

Refer to existing Pull Requests to have a better view about the work to do, for instance:
- [Error Event Rendering Pull Request](https://github.com/process-analytics/bpmn-visualization-js/pull/505/files)
Expand All @@ -121,3 +121,14 @@ a Java tool that will let you transform your SVG file into a set of `TypeScript`

Please be aware that the tool is not able to support all SVG files, and you may need to adapt the SVG definition prior the
tool can transform it. See [PR #210](https://github.com/process-analytics/bpmn-visualization-js/pull/210) for instance.


#### <a name="icons-license"></a> Reusing existing icons

If you integrate icons that you have not designed by yourself, please don't forget to credit its author and reproduction
conditions. Please try to use materials covered by a Free License to avoid any license compliance issues.

In that case, you must add credit in the following docs:
- in the source code
- in the BPMN support documentation: at the same place or close to the BPMN supported element
- in the main README: we don't list all icons there, but we reference projects where the icons come from
2 changes: 2 additions & 0 deletions docs/index.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
:sectlinks:
// number parts of a book
:partnums:
// include a link to a favicon
:favicon:
// enable font-awesome icons
:icons: font

Expand Down
13 changes: 7 additions & 6 deletions docs/intro.adoc
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
== Introduction
:favicon:

image::images/diagram-example.png[]

Expand Down Expand Up @@ -39,15 +40,15 @@ If you need BPMN examples, you can use resources from:
|===
| image:https://www.google.com/chrome/static/images/chrome-logo.svg[width=18] Chrome | image:https://user-media-prod-cdn.itsre-sumo.mozilla.net/uploads/products/2020-04-14-08-36-13-8dda6f.png[width=18] Firefox | image:https://developer.apple.com/assets/elements/icons/safari/safari-96x96.png[width=18] Safari | image:https://avatars0.githubusercontent.com/u/11354582?s=200&v=4[width=18] Edge

|Yes
|Yes
|Yes
|Yes
| icon:check[]
| icon:check[]
| icon:check[]
| icon:check[]
Comment on lines +43 to +46
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍


|===

**Note**: Internet Explorer won't never be supported. +
The library may work with the other browsers, currently not list, if they support ES6.
**Note**: Internet Explorer will never be supported. +
The library may work with the other browsers, they must at least support ES6.
tbouffard marked this conversation as resolved.
Show resolved Hide resolved


== ♻️ Usage
Expand Down
1 change: 1 addition & 0 deletions scripts/docs.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ fse.ensureDirSync(`${docsOutput}/images`);

fse.copySync('docs/images', `${docsOutput}/images`);
fse.copySync('docs/architecture/images', `${docsOutput}/images`);
fse.copySync('src/static/img/favicon.ico', `${docsOutput}/favicon.ico`);

// eslint-disable-next-line no-console
console.info(`Documentation is now available in ${docsOutput}`);