• Type: Feature Request
• Resolution: Fixed
• Priority: L3 - Default
• Fix Version/s: 1.0.0
• Affects Version/s: None
• Component/s: frontend
• Labels:

  None

This is the controller panel for Smart Panels app

[OPT-421] Process Definition is disabled if xml is not imported yet

Collapse comment: Sebastian Stamm added a comment - 29/May/17 11:40 AM

Sebastian Stamm added a comment - 29/May/17 11:40 AM

Could we use the Class component to toggle the display of the error and the viewer node in the diagram preview widget instead of writing to the style attribute directly?

Expand comment: Sebastian Stamm added a comment - 29/May/17 11:40 AM

Sebastian Stamm added a comment - 29/May/17 11:40 AM Could we use the Class component to toggle the display of the error and the viewer node in the diagram preview widget instead of writing to the style attribute directly?

Collapse comment: Sebastian Stamm added a comment - 01/Jun/17 5:17 PM

Sebastian Stamm added a comment - 01/Jun/17 5:17 PM

DiagramPreview should not rely on setElementVisibility util to always name the css class 'hidden'. We should instead call setElementVisibility to hide the element initially and offer an isElementVisible in the visibility utility instead of using hasClass in the DiagramPreview.

Expand comment: Sebastian Stamm added a comment - 01/Jun/17 5:17 PM

Sebastian Stamm added a comment - 01/Jun/17 5:17 PM DiagramPreview should not rely on setElementVisibility util to always name the css class 'hidden'. We should instead call setElementVisibility to hide the element initially and offer an isElementVisible in the visibility utility instead of using hasClass in the DiagramPreview.

Collapse comment: Sebastian Stamm added a comment - 02/Jun/17 11:16 AM

Sebastian Stamm added a comment - 02/Jun/17 11:16 AM

DiagramPreview still relies on the explicit name 'hidden' in the template. We should instead call setElementVisibility to hide the element initially.

Expand comment: Sebastian Stamm added a comment - 02/Jun/17 11:16 AM

Sebastian Stamm added a comment - 02/Jun/17 11:16 AM DiagramPreview still relies on the explicit name 'hidden' in the template. We should instead call setElementVisibility to hide the element initially.