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

Revert "Fix text anchor position when txt.Bounds().W() != txt.Dot.X-txt.Orig.X" #117

Merged
merged 1 commit into from
Aug 30, 2024

Conversation

bhperry
Copy link
Contributor

@bhperry bhperry commented Aug 30, 2024

Fix for #84

This reverts commit 3b599e7.

With the previous change in place, text was not being properly aligned to the Dot position
image

With that line reverted, the typewriter example works properly
image

Not entirely clear to me what the change was attempting to fix, but I have been unable to find a scenario where that offset calc is correct. Still need to do some work on text anchoring in general, as I don't think it is completely functional yet.

@bhperry bhperry requested a review from a team August 30, 2024 22:16
@bhperry bhperry merged commit 8f8ea78 into gopxl:main Aug 30, 2024
1 check passed
@bhperry bhperry deleted the revert-text-anchor-bug branch August 30, 2024 23:28
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

Successfully merging this pull request may close these issues.

2 participants