OWASP / threat-dragon

An open source threat modeling tool from OWASP
https://owasp.org/www-project-threat-dragon/
Apache License 2.0
927 stars 249 forks source link

I have to click undo five times to actually undo the action #1068

Closed srcnuzn closed 3 weeks ago

srcnuzn commented 2 months ago

Describe the bug:

When creating a data flow by double clicking a process, I have to execute undo five times in order to actually undo the action.

Expected behaviour:

Only clicking once should be enough.

Environment:

To Reproduce:

  1. Create Process Element.
  2. Double click on it.
  3. Click on undo five times.

Any additional context, screenshots, etc:

2024-08-22_13h19_02

jgadsden commented 2 months ago

wow, that it not a good look, I agree @srcnuzn and thanks again for the bug report, much appreciated There is work to be done for redo/undo functionality, along with #972 , planning this for version 2.3 due in Q3 2024

jgadsden commented 1 month ago

This is fixed in pull request 'Migrate to version 2.x antv/x6' #1099, where the undo functions as expected and the double click to add a dataflow has been removed - it was too counter intuitive