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

Misaligned routes when chrome zoom is not 100% #218

Closed
TLeiter opened this issue Jul 26, 2021 · 13 comments
Closed

Misaligned routes when chrome zoom is not 100% #218

TLeiter opened this issue Jul 26, 2021 · 13 comments
Assignees
Labels
enhancement New feature or request

Comments

@TLeiter
Copy link

TLeiter commented Jul 26, 2021

When user has a different zoom factor in Chrome, the routes are not alligned.

The following example was done with the demo application:

2021-07-26 09_22_15-Clipboard

@jerosoler jerosoler self-assigned this Jul 26, 2021
@jerosoler
Copy link
Owner

Uhmmm. What is your version of crhome?

image

@TLeiter
Copy link
Author

TLeiter commented Jul 26, 2021

Last one Version 92.0.4515.107 (Official Build) (64-bit)

@TLeiter
Copy link
Author

TLeiter commented Jul 26, 2021

Zoom at 150% works fine, but 125% not.

@jerosoler
Copy link
Owner

Work in 125%

image

This I think could be due to a browser problem rather than a library problem.

With the library you can set a zoom.

@Fr1411
Copy link

Fr1411 commented Aug 2, 2021

i have the same problem with chrome Versión 92.0.4515.107 (Build oficial).
No zoom.

image

zoom 80% its works
image

@jerosoler
Copy link
Owner

@Fr1411
If you move any node, are they still wrong?

I see the images of the demo.
The same thing happens without the demo.

@jerosoler
Copy link
Owner

I just installed the latest version and I see that it also happens.

image

@jerosoler jerosoler added the enhancement New feature or request label Aug 2, 2021
@Fr1411
Copy link

Fr1411 commented Aug 3, 2021

@jerosoler yes! if i move any node it´s the same.
In my project the situation is equal
image

@Fr1411
Copy link

Fr1411 commented Aug 4, 2021

@jerosoler Do you think this problem will be solved?

@Ammy22
Copy link

Ammy22 commented Aug 4, 2021

Temporary fix is
image

jerosoler added a commit that referenced this issue Aug 4, 2021
@jerosoler
Copy link
Owner

Fix in new version 0.0.48 https://github.com/jerosoler/Drawflow/releases/tag/0.0.48

Removed transform tranlate from connection.

The new css looks like this:

.drawflow .connection {
  position: absolute;
  pointer-events: none;
}
.drawflow .connection .main-path {
  fill: none;
  stroke-width: 5px;
  stroke: steelblue;
  pointer-events: all;
}
.drawflow .connection .main-path:hover {
  stroke: #1266ab;
  cursor: pointer;
}

.drawflow .connection .main-path.selected {
  stroke: #43b993;
}

.drawflow .connection .point {
  cursor: move;
  stroke: black;
  stroke-width: 2;
  fill: white;
  pointer-events: all;
}

Update Demo page

@Ammy22
Copy link

Ammy22 commented Aug 4, 2021 via email

@Fr1411
Copy link

Fr1411 commented Aug 4, 2021

Thank you!

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

No branches or pull requests

4 participants