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

Double clicking causes cloning #3262

Open
1 of 3 tasks
vanillajonathan opened this issue Apr 21, 2024 · 3 comments
Open
1 of 3 tasks

Double clicking causes cloning #3262

vanillajonathan opened this issue Apr 21, 2024 · 3 comments

Comments

@vanillajonathan
Copy link
Contributor

Describe the bug

Double clicking on an item in the model browser results in the item to be cloned and inserted into the diagram. I find this very unexpected and confusing.

To Reproduce

Steps to reproduce the behavior:

  1. Double click on an item in the model browser.
  2. Notice how it gets inserted into the diagram.

Expected behavior

I don't know, maybe it should offer to rename the item, maybe it should do nothing, but it definitely should not clone the item and insert it into the diagram.

OS

  • Linux (Ubuntu)
  • macOS
  • Windows

Version

Version of Gaphor: 2.25.0

@tompkins-ct
Copy link

tompkins-ct commented Apr 22, 2024

I like that I can double-click on a relationship and it shows up in the current diagram (if the endpoints are present). I can't say I've used the double-click on other items to insert them.

However, the item doesn't get cloned (at least for block definition diagrams); it just gets added to the current diagram. If you right-click on an item, it shows all the diagrams that it's present on in your model.

@danyeaw
Copy link
Member

danyeaw commented Apr 25, 2024

Hi @vanillajonathan, thanks for the issue report! As @tompkins-ct said, I don't think it clones the item (creates another copy), it simply puts a new instance of the object on the diagram.

What if double clicking selected the object on the diagram, or gave a message that the current element isn't on the current diagram if it doesn't exist?

@vanillajonathan
Copy link
Contributor Author

Alright but I don't like that it puts a new instance of the object on the diagram, I found it unexpected and confusing.

Yeah, I think one of those options would be better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants