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

TranslateTool.py update setter and getter to allow for Y axis flip to show expected values in UI for Y #983

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

HellAholic
Copy link
Contributor

@HellAholic HellAholic commented Dec 7, 2024

Add a direction variable that changes to -1 if the y axis if flipped in the preferences.
This will put the values shown in the UI in line with the direction of the tool handle flip (Y+ is in direction that the toolhandle points to)
Ultimaker/Cura#20006

Add a direction variable that changes to -1 if the y axis if flipped in the preferences.
Copy link

github-actions bot commented Dec 7, 2024

Test Results

2 407 tests  ±0   2 392 ✅ ±0   22s ⏱️ -1s
    1 suites ±0      15 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 2f843e6. ± Comparison against base commit 1d021be.

@HellAholic HellAholic changed the title Update setter and getter TranslateTool.py update setter and getter to allow for Y axis flip to show expected values in UI for Y Dec 7, 2024
@GregValiant
Copy link
Contributor

I'm catching up on my reviews. It's been about 3 weeks now and I haven't had any problems with this at all. Same with the changes to the Rotation Tool.

@HellAholic HellAholic merged commit a986eba into main Jan 23, 2025
5 checks passed
@HellAholic HellAholic deleted the translatetool_flip_y branch January 23, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Community Contribution 👑 Community Contribution PR's
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants