@camunda/linting 0.5.0-alpha.1 → 0.5.0

package/assets/linting.css

@@ -0,0 +1,33 @@
+:root {
+  --color-red-360-100-45: hsl(360, 100%, 45%);
+  --color-white: hsl(0, 0%, 100%);
+
+  --linting-annotation-background-color: var(--color-red-360-100-45);
+  --linting-annotation-fill-color: var(--color-white);
+}
+
+.djs-overlays:not(.selected) .bjs-linting-annotation {
+  opacity: 0.5;
+}
+
+.djs-overlays.selected .bjs-linting-annotation,
+.djs-overlays .bjs-linting-annotation:hover {
+  opacity: 1;
+}
+
+.bjs-linting-annotation {
+  display: flex;
+  flex-direction: row;
+  align-items: center;
+  padding: 1px;
+  border-radius: 2px;
+  background-color: var(--linting-annotation-background-color);
+  cursor: default;
+  z-index: 100000;
+}
+
+.bjs-linting-annotation > svg {
+  fill: var(--linting-annotation-fill-color);
+  width: 18px;
+  height: 18px;
+}

package/lib/modeler/Linting.js

@@ -1,13 +1,14 @@
 import { getErrors } from '../utils/properties-panel';
 
 export default class Linting {
-  constructor(canvas, elementRegistry, eventBus, lintingAnnotations, selection) {
+  constructor(canvas, config, elementRegistry, eventBus, lintingAnnotations, selection) {
     this._canvas = canvas;
     this._elementRegistry = elementRegistry;
     this._eventBus = eventBus;
     this._lintingAnnotations = lintingAnnotations;
     this._selection = selection;
 
+    this._active = config && config.active || false;
     this._reports = [];
 
     eventBus.on('selection.changed', () => this._update());
@@ -43,20 +44,25 @@ export default class Linting {
   }
 
   activate() {
+    this._active = true;
+
     this._update();
   }
 
   deactivate() {
-    this._update([]);
+    this._active = false;
+
+    this._update();
   }
 
-  _update(reports) {
-    if (!reports) {
-      reports = this._reports;
-    }
+  isActive() {
+    return this._active;
+  }
+
+  _update() {
 
     // set annotations
-    this._lintingAnnotations.setErrors(reports);
+    this._lintingAnnotations.setErrors(this.isActive() ? this._reports : []);
 
     // set properties panel errors
     const selectedElement = this._getSelectedElement();
@@ -85,6 +91,7 @@ export default class Linting {
 
 Linting.$inject = [
   'canvas',
+  'config.linting',
   'elementRegistry',
   'eventBus',
   'lintingAnnotations',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camunda/linting",
-  "version": "0.5.0-alpha.1",
+  "version": "0.5.0",
   "description": "Linting for Camunda Platform",
   "main": "index.js",
   "scripts": {
@@ -62,6 +62,7 @@
     }
   },
   "files": [
+    "assets",
     "lib",
     "modeler.js"
   ],

package/CHANGELOG.md

@@ -1,66 +0,0 @@
-# Changelog
-
-All notable changes to [@camunda/linting](https://github.com/camunda/linting) are documented here. We use [semantic versioning](http://semver.org/) for releases.
-
-## Unreleased
-
-___Note:__ Yet to be released changes appear here._
-
-## 0.5.0-alpha.1
-
-* `FIX`: publish modeler.js
-
-## 0.5.0-alpha.0
-
-* `FEAT`: add bpmn-js plugin for canvas and properties panel errors ([#11](https://github.com/camunda/linting/pull/11))
-* `FEAT`: add no-zeebe-properties rule ([#43](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/43))
-
-## 0.4.1
-
-* `DEPS`: broaden supported versions range of `bpmn-js-properties-panel`
-
-## 0.4.0
-
-* `FEAT`: add duplicate task headers rule ([#41](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/41))
-
-## 0.3.5
-
-* `FIX`: ignore null properties ([#39](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/39))
-
-## 0.3.4
-
-* `FIX`: make `bpmn-moddle` and `zeebe-bpmn-moddle` production dependencies ([#9](https://github.com/camunda/linting/pull/9))
-
-## 0.3.3
-
-* `FIX`: fix business rule task error message in properties panel ([#8](https://github.com/camunda/linting/pull/8))
-
-## 0.3.2
-
-* `FIX`: make `modeler-moddle` dependency ([e9d53714](https://github.com/camunda/linting/commit/e9d5371456cd2e783ae2c7c0c3ca0f4c5047db1a))
-
-## 0.3.1
-
-* `FIX`: include `properties-panel.js` in published package ([a532ba5b](https://github.com/camunda/linting/commit/a532ba5b7bf0b126477c218484e668c418875b4e))
-
-## 0.3.0
-
-* `FEAT`: add properties panel entry ID to reports ([#7](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/7))
-* `FEAT`: add #getErrors function that creates properties panel errors from reports ([#7](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/7))
-* `FEAT`: adjust connectors error message ([#6](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/6))
-
-### BREAKING CHANGES
-
-* #lint is not static anymore, Linter must be instantiated
-
-## 0.2.0
-
-* `FEAT`: add templates rule ([#31](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/31))
-
-## 0.1.1
-
-* `FIX`: lint subscription only if start event child of sub process ([#34](https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/34))
-
-## 0.1.0
-
-* `FEAT`: initial release bundling [bpmnlint](https://github.com/bpmn-io/bpmnlint) and [bpmnlint-plugin-camunda-compat](https://github.com/camunda/bpmnlint-plugin-camunda-compat/) and Camunda specific functionality ([#1](https://github.com/camunda/linting/pull/1))