Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve context pad tooltip for exclusive gateway #917

Closed
mschoe opened this issue Sep 24, 2018 · 3 comments
Closed

Improve context pad tooltip for exclusive gateway #917

mschoe opened this issue Sep 24, 2018 · 3 comments
Assignees
@pinussilvestrus
Labels
BPMN enhancement New feature or request good first issue Good for newcomers modeling
Milestone
M25

Comments

@mschoe
Copy link
Member

mschoe commented Sep 24, 2018

Given
An event based gateway

When
I use the context menu to append an intermediate catching event and hover over the event symbols

Then
I get the same tooltip for all events: Append IntermediateCatchEvent

Expected Behavior
The tooltip contains the exact event type (Signal, Message, etc.)
@nikku nikku added good first issue Good for newcomers BPMN modeling enhancement New feature or request labels Sep 24, 2018
@nikku
Copy link
Member

nikku commented Sep 24, 2018

This needs a fix in bpmn-js, our underlying modeling toolkit.

@nikku nikku added the ready Ready to be worked on label Sep 24, 2018
@nikku nikku changed the title Tooltip Improve context pad tooltip for exclusive gateway Sep 26, 2018
@pinussilvestrus pinussilvestrus added in progress Currently worked on and removed ready Ready to be worked on labels Oct 1, 2018
@pinussilvestrus pinussilvestrus self-assigned this Oct 1, 2018
@pinussilvestrus pinussilvestrus mentioned this issue Oct 1, 2018
Merged
@pinussilvestrus pinussilvestrus added needs review Review pending and removed in progress Currently worked on labels Oct 1, 2018
@nikku
Copy link
Member

nikku commented Oct 2, 2018

Pending bpmn-js release.

@nikku nikku added this to the M25 milestone Oct 4, 2018
@pinussilvestrus pinussilvestrus mentioned this issue Nov 5, 2018
Closed
@nikku
Copy link
Member

nikku commented Nov 19, 2018

Closed via 00f3ec8.

@nikku nikku closed this as completed Nov 19, 2018
@ghost ghost removed the needs review Review pending label Nov 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pinussilvestrus pinussilvestrus

Labels
BPMN enhancement New feature or request good first issue Good for newcomers modeling
Projects
None yet
Milestone
M25
Development

No branches or pull requests
3 participants
@nikku @mschoe @pinussilvestrus