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

Proxy port and activity pin stay connected; activity parameter node is always stuck to the activity when moving #2542

Merged
merged 3 commits into from
Jul 27, 2023

Conversation

marek-piirikivi
Copy link
Contributor

@marek-piirikivi marek-piirikivi commented Jul 24, 2023

I added StickyAttachedHandleMove which can be registered to the items that once connected, can not be disconnected via dragging and can only be moved along the border.

I added this behavior to activity parameters (only dragging functionality; otherwise was already there inability to disconnect); pins (both input and output); proxyport.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bug fix
  • Feature
  • Chore (refactoring, formatting, local variables, other cleanup)
  • Documentation content changes

What is the current behavior?

Issue Number: #2484, #2450 (only point 3 from the initially proposed solution)

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

From the both related issues, not having free floating port/pin at all is not yet addressed - it is still possible to have free floating port/pin if it has never been attached to anything, but at least it is not possible to detach it anymore other than explicitly deleting.

… connection; ensure that activity parameter node is always stuck to the activity when moving.
@github-actions github-actions bot added the python Pull requests that update Python code label Jul 24, 2023
@sz332 sz332 requested a review from amolenaar July 26, 2023 07:17
@marek-piirikivi marek-piirikivi changed the title Disallow disconnection by dragging for proxy port, activity pin after… Disallow disconnection by dragging for proxy port, activity pin after connection; ensure that activity parameter node is always stuck to the activity when moving. Jul 26, 2023
amolenaar and others added 2 commits July 26, 2023 22:03
The blockmove module indirectly import Gdk. That should only
happen if a uit session is loaded.
Move blockmove initialization to uicomponents module
@amolenaar
Copy link
Member

Works like a charm. Thanks @marek-piirikivi!

@amolenaar amolenaar merged commit 4e4b1d5 into gaphor:main Jul 27, 2023
20 checks passed
@amolenaar amolenaar changed the title Disallow disconnection by dragging for proxy port, activity pin after connection; ensure that activity parameter node is always stuck to the activity when moving. Proxy port and activity pin stay connected; activity parameter node is always stuck to the activity when moving Jul 27, 2023
@marek-piirikivi marek-piirikivi deleted the sticky_port_pin_parameter branch July 28, 2023 03:47
@danyeaw danyeaw added fix A fix for a bug and removed python Pull requests that update Python code labels Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix A fix for a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants