Adding Color to Bpmn Modeler creates invalid XML

Hi Guys,

So as the title says, I have added color , as showed in the repo bpmn-js colors to my modeler.
Everything seemed to be fine, but I just noticed than when trying to save the diagram, this produces invalid xml, therefore not being able to re-open the saved file in the modeler, or any BPMN tool.
Just wondering why would this happen? And if it can be fixed?

Thank you,

If it helps here is the invalid xml produced by the app:

<?xml version="1.0" encoding="UTF-8"?>
<bpmn2:definitions xmlns:xsi="" xmlns:bpmn2="" xmlns:bpmndi="" xmlns:dc="" xmlns:di="" xmlns:bioc="" id="sample-diagram" targetNamespace="" xsi:schemaLocation=" BPMN20.xsd">
  <bpmn2:process id="Process_1" isExecutable="false">
    <bpmn2:startEvent id="StartEvent_1">
      <bpmn2:documentation textFormat="text/x-comments"></bpmn2:documentation>
    <bpmn2:task id="Task_0kc3hyc">
    <bpmn2:sequenceFlow id="SequenceFlow_1d2aw04" sourceRef="StartEvent_1" targetRef="Task_0kc3hyc" />
  <bpmndi:BPMNDiagram id="BPMNDiagram_1">
    <bpmndi:BPMNPlane id="BPMNPlane_1" bpmnElement="Process_1">
      <bpmndi:BPMNShape id="_BPMNShape_StartEvent_2" bpmnElement="StartEvent_1" bioc:fill="">
        <dc:Bounds x="280" y="239" width="36" height="36" />
          <dc:Bounds x="253" y="275" width="90" height="20" />
      <bpmndi:BPMNShape id="Task_0kc3hyc_di" bpmnElement="Task_0kc3hyc" bioc:fill="" $type="bpmndi:BPMNShape">
        <dc:Bounds x="554" y="217" width="100" height="80" />
      <bpmndi:BPMNEdge id="SequenceFlow_1d2aw04_di" bpmnElement="SequenceFlow_1d2aw04">
        <di:waypoint x="316" y="257" />
        <di:waypoint x="554" y="257" />
          <dc:Bounds x="435" y="236" width="0" height="12" />

I’ve updated the example to the latest bpmn-js version. That should fix the issues you’re having.

I’d suggest you to use our built-in color support instead, as the example builds on a dead BPMN standard proposal.

Thank you for that @nikku . That did solve the problem indeed.
I will look at the built-in color support if that is what are you suggesting is a better approach.