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

Pressing <DEL> in properties panel deletes model element #680

Closed
nikku opened this issue Oct 6, 2017 · 1 comment
Closed

Pressing <DEL> in properties panel deletes model element #680

nikku opened this issue Oct 6, 2017 · 1 comment
Assignees
@nikku
Labels
bug Something isn't working channel:support ux
Milestone
M22

Comments

@nikku
Copy link
Member

nikku commented Oct 6, 2017 •
edited by mburke84
Loading

Context

  • BPMN, may be applicable to CMMN as well

Reproduce

  • Model Task
  • Open Properties Panel
  • Switch to Listener Tab
  • Press + (add listener)
  • Press DEL

Task is gone.

Expected Behavior

  • Nothing happens or selected listener is deleted

Also see SUPPORT-3700
@nikku nikku added bug Something isn't working ux labels Oct 6, 2017
@nikku nikku added this to the Backlog milestone Oct 6, 2017
@nikku nikku added the backlog Queued in backlog label Feb 14, 2018
@nikku nikku removed this from the Backlog milestone Feb 14, 2018
@nikku nikku added this to the M21 milestone Apr 23, 2018
@nikku
Copy link
Member Author

nikku commented Apr 23, 2018

We'll look into this as a spike.

@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Apr 23, 2018
@nikku nikku modified the milestones: M21, M22 Apr 26, 2018
@philippfromme philippfromme added in progress Currently worked on ready Ready to be worked on and removed ready Ready to be worked on in progress Currently worked on labels May 18, 2018
@nikku nikku modified the milestones: M22, M23 May 28, 2018
@philippfromme philippfromme added in progress Currently worked on and removed ready Ready to be worked on labels May 28, 2018
@philippfromme philippfromme self-assigned this May 28, 2018
philippfromme added a commit to bpmn-io/bpmn-js-properties-panel that referenced this issue May 28, 2018
@philippfromme
fix(properties-panel): stop propagation of key events
94e01cb 
Related to camunda/camunda-modeler#680
@philippfromme philippfromme added needs review Review pending and removed in progress Currently worked on labels May 28, 2018
@philippfromme philippfromme assigned nikku and unassigned philippfromme May 28, 2018
@nikku nikku modified the milestones: M23, M22 May 28, 2018
@philippfromme philippfromme added in progress Currently worked on and removed needs review Review pending labels May 28, 2018
philippfromme added a commit to bpmn-io/bpmn-js-properties-panel that referenced this issue May 28, 2018
@philippfromme
fix(properties-panel): stop propagation of DEL key events
71dc3a4 
Related to camunda/camunda-modeler#680
@philippfromme philippfromme added needs review Review pending and removed in progress Currently worked on labels May 28, 2018
@nikku nikku closed this as completed in 49c96c3 May 29, 2018
@ghost ghost removed the needs review Review pending label May 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nikku nikku

Labels
bug Something isn't working channel:support ux
Projects
None yet
Milestone
M22
Development

No branches or pull requests
3 participants
@nikku @mburke84 @philippfromme