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

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

Closed
srcnuzn opened this issue Aug 22, 2024 · 2 comments · Fixed by #1099
Closed

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

srcnuzn opened this issue Aug 22, 2024 · 2 comments · Fixed by #1099
Assignees
Labels
AntV/X6 bug Something isn't working version-2.3
Milestone

Comments

@srcnuzn
Copy link

srcnuzn commented Aug 22, 2024

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:

  • Version: 2.2
  • Platform: Desktop App
  • OS: Windows 11

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

@srcnuzn srcnuzn added the bug Something isn't working label Aug 22, 2024
@jgadsden
Copy link
Collaborator

jgadsden commented Aug 23, 2024

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
Copy link
Collaborator

jgadsden commented Oct 5, 2024

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

@jgadsden jgadsden self-assigned this Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AntV/X6 bug Something isn't working version-2.3
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants