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.

Sign up for GitHub

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

Wrong layering of context pad in DRD editing #657

Closed
volkergersabeck opened this issue Aug 26, 2021 · 1 comment · Fixed by #670
Closed

Wrong layering of context pad in DRD editing #657

volkergersabeck opened this issue Aug 26, 2021 · 1 comment · Fixed by #670
Assignees
@philippfromme
Labels
bug Something isn't working spring cleaning Could be cleaned up one day ux

Comments

@volkergersabeck
Copy link

Describe the Bug

When putting decision table nodes in the DRD view quite close to each other
then the context pad is overlapped by the blue decision table icon
and certain icons of the context pad cannot be used anymore

Steps to Reproduce

See screenshot:

Screenshot 2021-08-25 at 10 57 16

Expected Behavior

The context pad should always be on top of other diagram overlays.

Environment

Observed in Cawemo, confirmed in Modeler 4.9.0
@volkergersabeck volkergersabeck added the bug Something isn't working label Aug 26, 2021
@MaxTru
Copy link
Contributor

MaxTru commented Aug 26, 2021

Can reproduce, moving to backlog

@MaxTru MaxTru added spring cleaning Could be cleaned up one day backlog Queued in backlog ux labels Aug 26, 2021
@philippfromme philippfromme self-assigned this Oct 14, 2021
philippfromme added a commit to bpmn-io/diagram-js that referenced this issue Oct 14, 2021
@philippfromme
fix(context-pad): ensure context pad is always in front
8348516 
Related to bpmn-io/dmn-js#657
@philippfromme philippfromme mentioned this issue Oct 14, 2021
Closed
nikku pushed a commit to bpmn-io/diagram-js that referenced this issue Oct 14, 2021
@philippfromme @nikku
fix(context-pad): ensure context pad is always in front
370ecb9 
Related to bpmn-io/dmn-js#657
@philippfromme philippfromme added the fixed upstream Requires integration of upstream change label Oct 15, 2021
@smbea smbea removed the backlog Queued in backlog label Jan 17, 2022
barmac added a commit that referenced this issue Jan 25, 2022
@barmac
deps: update to [email protected]
c5d7d5b 
Closes #657
@barmac barmac mentioned this issue Jan 25, 2022
Merged
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed fixed upstream Requires integration of upstream change labels Jan 25, 2022
@fake-join fake-join bot closed this as completed in #670 Jan 25, 2022
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jan 25, 2022
fake-join bot pushed a commit that referenced this issue Jan 25, 2022
@barmac @fake-join
deps: update to [email protected]
ee0f067 
Closes #657
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philippfromme philippfromme

Labels
bug Something isn't working spring cleaning Could be cleaned up one day ux
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

deps: update to [email protected] bpmn-io/dmn-js
4 participants
@volkergersabeck @philippfromme @smbea @MaxTru