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

[BUG] Ellipse Guidelines Are Not Correct with Offsets #395

Closed
RyanR712 opened this issue May 9, 2024 · 1 comment · Fixed by #396
Closed

[BUG] Ellipse Guidelines Are Not Correct with Offsets #395

RyanR712 opened this issue May 9, 2024 · 1 comment · Fixed by #396
Assignees
Labels
Bug Something isn't working. Rightmost number in semantic versioning good first issue Good for newcomers User Interface What the user sees

Comments

@RyanR712
Copy link
Collaborator

RyanR712 commented May 9, 2024

Describe the bug
Ellipse guidelines are not correctly calculated when an offset is introduced.

To Reproduce
Use the drag tool to introduce an offset.

Click the "Show Guidelines" checkbox for ellipses.

Draw an ellipse. You will see that the offset is not taken into consideration.

Expected behavior
Ellipse guidelines should be around each ellipse, regardless of offset.

Screenshots Or Video

Ellipse.Guideline.Offset.Issue.mp4
@RyanR712 RyanR712 added Bug Something isn't working. Rightmost number in semantic versioning good first issue Good for newcomers User Interface What the user sees labels May 9, 2024
@James-Oswald
Copy link
Member

Interesting, I guess the bounding boxes aren't being moved by the offset, hopefully this issue is as simple as adding the offset to their positions.

@RyanR712 RyanR712 self-assigned this May 9, 2024
@RyanR712 RyanR712 linked a pull request May 9, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working. Rightmost number in semantic versioning good first issue Good for newcomers User Interface What the user sees
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants