Skip to content

fix(modeler): keep default task marker visible with template icon#98

Open
ivanfarray-cib wants to merge 1 commit into
mainfrom
tweaks-iconrender
Open

fix(modeler): keep default task marker visible with template icon#98
ivanfarray-cib wants to merge 1 commit into
mainfrom
tweaks-iconrender

Conversation

@ivanfarray-cib
Copy link
Copy Markdown
Member

Render the applied element-template icon in the opposite (top-right) corner on activities so the default task-type marker (user figure on user tasks, gears on service tasks, ...) stays visible as a reference instead of being replaced. Events still replace-and-center because a 36 px circle has no room for two icons.

Also fix the bpmn-js handler lookup: the .find() over base types returns 'bpmn:Task' first for a UserTask, and handlers['bpmn:Task'] is the plain task handler with no marker. Prefer handlers[element.type] and fall back to baseType only for custom subclasses.

Render the applied element-template icon in the opposite (top-right)
corner on activities so the default task-type marker (user figure on
user tasks, gears on service tasks, ...) stays visible as a reference
instead of being replaced. Events still replace-and-center because a
36 px circle has no room for two icons.

Also fix the bpmn-js handler lookup: the .find() over base types
returns 'bpmn:Task' first for a UserTask, and handlers['bpmn:Task']
is the plain task handler with no marker. Prefer handlers[element.type]
and fall back to baseType only for custom subclasses.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants