pycodestyle icon indicating copy to clipboard operation
pycodestyle copied to clipboard

Bug: Group can be dragged even when container is locked

Open jshannon-tml opened this issue 2 years ago • 1 comments

When I add the locked prop to the Svelvet container the Group can still be dragger.

To reproduce:

  1. Create a Svelvet container, add nodes, add a group with some more nodes.
  2. Add locked prop to container

Behavior: Hovering over the nodes shows the "no" cursor, and I cannot drag them. This includes nodes inside of the container. However, hovering over the Group, the cursor doesn't change and if I click-and-drag then the group (and its nodes) is moved.

Expectation: Shouldn't be able to move the Group.

I think that, since Groups can be quite large and take up a lot of the container, then they shouldn't have the same behavior as the node by preventing any dragging at all. Instead I think that the dragging should be allowed but should be used for panning instead. But that'd be more consistent with empty space than with nodes. So... hard to say.

jshannon-tml avatar May 08 '23 05:05 jshannon-tml

We'll have this fixed in the next release. I think your behavior suggestion is a good one.

briangregoryholmes avatar May 11 '23 00:05 briangregoryholmes