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

Block-dynamic-connection plugin undo does not resize dynamic create text block #2434

Open
1 task done
changminbark opened this issue Aug 5, 2024 · 2 comments
Open
1 task done
Labels
type: bug Something isn't working

Comments

@changminbark
Copy link
Contributor

Check for duplicates

  • I have searched for similar issues before opening a new one.

Component

Block-dynamic-connection plugin

Description

The dynamic create text block does not shrink back to its original number when I undo the action of dynamically connecting the block. If I move it off using the cursor, it resizes properly. It just doesn't do it with the undo. Refer to the gif to see what I am talking about.

Reproduction steps

Stack trace

No response

Screenshots

dynamic block bug

@cpcallen
Copy link
Contributor

cpcallen commented Aug 9, 2024

Hi @changminbark. Can you provide exact steps to reproduce this, please?

I tried to reproduce it as follows and got a different incorrect undo behaviour:

  1. Go to test page for dynamic connection plugin.
  2. Drag a "create text with" (join) block from the toolbox to the workspace.
  3. Drag an "abc" block to the first input of the join block.
  4. Drag an "abc" block to the second input of the join block.
  5. Drag an "abc" block to join block to get a third input to appear, and attach the "abc" block to it.
  6. Press undo repeatedly.
  7. Observe that the first undo works as expected, removing the third "abc" block. It then takes several undos to fully undo the addition of the second "abc" block. The first "abc" block is undone in a single undo as expected.

There is at least one bug here but I'm not sure if it's the one that @changminbark reported.

@cpcallen cpcallen removed the triage label Aug 9, 2024
@changminbark
Copy link
Contributor Author

Here are the exact steps I took to produce the behavior:

  1. Go to test page for dynamic connection plugin.
  2. Drag a "create text with" (join) block from the toolbox to the workspace.
  3. Drag an "abc" block to the workspace.
  4. Connect this to first input of the join block.
  5. Drag an "def" block to the workspace.
  6. Connect this to second input of the join block.
  7. Drag an "ghi" block to the workspace.
  8. Connect this to third input of the join block. (This will dynamically create the third input)
  9. Press undo, disconnecting the third input but not deleting the input "ghi" block.

If you follow this step, you will end up with a "create text with" (join) block with 3 inputs even though there are only 2 input blocks:
image

I am not familiar with the other bug. It seems that these bugs are fundamentally different as I was concerned with the "create text with" (join) block not destroying the third input when the undo action is performed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants