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

Properties panel entry accidentally focuses #4710

Open
nikku opened this issue Nov 20, 2024 · 0 comments
Open

Properties panel entry accidentally focuses #4710

nikku opened this issue Nov 20, 2024 · 0 comments
Labels
backlog Queued in backlog BPMN bug Something isn't working properties panel spring cleaning Could be cleaned up one day

Comments

@nikku
Copy link
Member

nikku commented Nov 20, 2024

Describe the bug

As a user I want to reliably be able to undo a longer chain of commands, and potentially redo them afterwards. What I found is that under some circumstances the properties panel would focus some updated entries forcibly as part of an undo/redo operation, hence breaking the undo/redo flow:

capture LkUpjD_optimized

Steps to reproduce

  1. Create a C8 diagram
  2. Model a single execution listener (on the process)
  3. Fill in listener fields
  4. Focus canvas, mass undo (keep CtrlOrCmd + Z pressed)
  5. See that Name input is focussed, and redo does not work

Expected behavior

The properties panel does not forcibly focus a field on undo/redo.

Environment

  • OS: Ubuntu 24 (Linux)
  • Camunda Modeler Version: v5.29.0
  • Execution Platform: [e.g. Camunda 7, Camunda 8]
  • Installed plug-ins: None

Additional context

The same thing cannot be reproduced in a reasonably long Camunda 7 configuration:

capture 79wi81_optimized

It cannot be reproduced with zeebe:Property, aka extension elements, too:

capture 0yWDR0_optimized

It cannot be reproduced with IO mappings either, so this may be an execution listener exclusive bug:

capture lM1z4K_optimized

@nikku nikku added bug Something isn't working properties panel BPMN labels Nov 20, 2024
@philippfromme philippfromme added backlog Queued in backlog spring cleaning Could be cleaned up one day labels Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog BPMN bug Something isn't working properties panel spring cleaning Could be cleaned up one day
Projects
None yet
Development

No branches or pull requests

2 participants