fix: handle labels correctly

Author: barmac

src/render/BpmnPropertiesPanelRenderer.js

@@ -94,7 +94,14 @@ export default class BpmnPropertiesPanelRenderer {
       if (selectedElements.length > 1) {
         this._selectedElement = selectedElements;
       } else if (selectedElements.length === 1) {
-        this._selectedElement = selectedElements[0];
+        let newSelectedElement = selectedElements[0];
+
+        // handle labels
+        if (newSelectedElement.type === 'label') {
+          newSelectedElement = newSelectedElement.labelTarget;
+        }
+
+        this._selectedElement = newSelectedElement;
       } else {
         this._selectedElement = rootElement;
       }

test/spec/BpmnPropertiesPanelRenderer.spec.js

@@ -387,6 +387,29 @@ describe('<BpmnPropertiesPanelRenderer>', function() {
     });
 
 
+    it('should rerender on selection changed (label)', async function() {
+
+      // given
+      const diagramXml = require('test/fixtures/simple.bpmn').default;
+
+      const { modeler } = await createModeler(diagramXml);
+
+      const spy = sinon.spy();
+
+      modeler.on('propertiesPanel.updated', spy);
+      const label = modeler.get('elementRegistry').get('StartEvent_1').label;
+
+      // when
+      await act(() => {
+        modeler.get('selection').select(label);
+      });
+
+      // then
+      expect(spy).to.have.been.called;
+      expect(domQuery('.bio-properties-panel-header-type', propertiesContainer).textContent).to.equal('Start Event');
+    });
+
+
     it('should rerender on providers changed', async function() {
 
       // given