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

Wrong link appearance while creating a link if zoom is activated #470

Closed
pierre-moire opened this issue Nov 20, 2019 · 17 comments
Closed

Wrong link appearance while creating a link if zoom is activated #470

pierre-moire opened this issue Nov 20, 2019 · 17 comments

Comments

@pierre-moire
Copy link
Contributor

pierre-moire commented Nov 20, 2019

I am pointing at the second "Test 1" here with a zoom of around 60.

image

The issue is probably introduced by the recent bugfix of #469 concerning link placement offset

EDIT : Added more details

@mathiasbaldissera
Copy link
Contributor

I'll check this

@mathiasbaldissera
Copy link
Contributor

Yes, it's a bug generated in the #469 bugfix, we forget to consider the zoom level in the calculation. @antonioru any idea of how to resolve this?

@mathiasbaldissera
Copy link
Contributor

I fixed the bug. I'll open a new PR

@mathiasbaldissera mathiasbaldissera mentioned this issue Nov 20, 2019
5 tasks
@antonioru
Copy link
Contributor

I fixed the bug. I'll open a new PR

Thank you Mathias, I wasn't able to read your messages until now.
I apologise

@pierre-moire
Copy link
Contributor Author

pierre-moire commented Nov 22, 2019

I just noticed that it still has issues.

If you just click on the start port without moving the mouse, the link appear but the position is incorrect. When we start moving the cursor, the end position of the link is then ok

image

EDIT: It doesn't sound systematic

@mathiasbaldissera
Copy link
Contributor

@pierre-moire can u tell me the exactly steps to reproduce the bug? I've tried here and it's work fine

@pierre-moire
Copy link
Contributor Author

I just zoom out then click on a out port to start creating a link. The mouse should not move at all when we click

@pierre-moire
Copy link
Contributor Author

Note that I am not sure if it's related to my own implementation. Feel free to close the ticket if you can't reproduce it

@mathiasbaldissera
Copy link
Contributor

I just zoom out then click on a out port to start creating a link. The mouse should not move at all when we click

@antonioru can u replicate this bug? I tried here But failed to replicate

@mathiasbaldissera
Copy link
Contributor

mathiasbaldissera commented Nov 23, 2019

Note that I am not sure if it's related to my own implementation. Feel free to close the ticket if you can't reproduce it

@pierre-moire Can u share your code?

@pierre-moire
Copy link
Contributor Author

pierre-moire commented Nov 23, 2019

I reproduced it on the demo itself (dynamic ports for instance).

I add a port then:

  • mouse over the port
  • click and maintain the mouse button clicked without moving the pointer at all
  • release the mouse button
  • the 2nd point of the link gets 0, 0 as coordinates, which lead to invalid representation of the link

image

image

@antonioru
Copy link
Contributor

I just zoom out then click on a out port to start creating a link. The mouse should not move at all when we click

@antonioru can u replicate this bug? I tried here But failed to replicate

Yes I do!
There's a bug there, basically if you click on a port without dragging a new link it creates a nonsense link segment.
I don't think this is a bug we introduced with last merges but I'll take a took at it

@antonioru
Copy link
Contributor

Here's the bug in all its vivid majesty

@antonioru
Copy link
Contributor

Also @pierre-moire I don't think this is related to the zoom lv.
Can you please correct the issue title?

@mathiasbaldissera
Copy link
Contributor

mathiasbaldissera commented Nov 23, 2019

Also @pierre-moire I don't think this is related to the zoom lv.
Can you please correct the issue title?

@antonioru He needs to open another issue, the zoom description is from another bug already fixed
@pierre-moire please, open another issue, close and put a reference This one

@antonioru
Copy link
Contributor

folks I've created the issue:
#481

@pierre-moire please close this one

@pierre-moire
Copy link
Contributor Author

Sure, thanks!

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