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

Different units for vertical & horizontal rulers #83

Closed
Amerlander opened this issue Jul 18, 2023 · 5 comments
Closed

Different units for vertical & horizontal rulers #83

Amerlander opened this issue Jul 18, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@Amerlander
Copy link

Amerlander commented Jul 18, 2023

Environments

Description

Using different units on vertical and horizontal rulers results in "switched" snapping of the guides.
The horizontal guides are fitting to the horizontal ruler, but they should fit the vertical ruler, since they are positioned vertical.

Lines placed on each snaping position:
image

@daybrush daybrush added question Further information is requested bug Something isn't working and removed question Further information is requested labels Jul 21, 2023
@daybrush
Copy link
Owner

@Amerlander

I will add the zoom value adjustment only for the guide line.

daybrush added a commit that referenced this issue Jul 22, 2023
@daybrush
Copy link
Owner

@Amerlander

moveable's new version is released.

Added guidesZoom property.

If not set or used as 0, it is automatically set to the same value as zoom.

@Amerlander
Copy link
Author

@daybrush Thank you for the update. It's not working properly yet; when I drop a horizontal guide at 0.5, it gets placed at 4 instead.

It might also be beneficial to introduce a guidesUnit option to specify the behavior of the guides in detail.

https://svelte.dev/repl/69716d51b0fd498f94ebcba9a57cee25?version=4.0.5

Video.mp4

daybrush added a commit that referenced this issue Jul 23, 2023
@daybrush
Copy link
Owner

@Amerlander

guides' new version is released. Check it agian.

@Amerlander
Copy link
Author

@daybrush thank you, works great now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants