Skip to content

Fix multi-line TextShape rotation#7404

Merged
emilk merged 2 commits intoemilk:mainfrom
afishhh:fix-text-rotation
Aug 5, 2025
Merged

Fix multi-line TextShape rotation#7404
emilk merged 2 commits intoemilk:mainfrom
afishhh:fix-text-rotation

Conversation

@afishhh
Copy link
Copy Markdown
Contributor

@afishhh afishhh commented Aug 2, 2025

#5411 broke rotation of multi-line TextShapes because PlacedRow::pos was no longer being rotated, so let's rotate it.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 2, 2025

Preview available at https://egui-pr-preview.github.io/pr/7404-fix-text-rotation
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin lucasmerlin added this to the 0.32.1 milestone Aug 5, 2025
@lucasmerlin lucasmerlin added bug Something is broken epaint labels Aug 5, 2025
Co-authored-by: Lucas Meurer <[email protected]>
@emilk emilk merged commit 31eb4d4 into emilk:main Aug 5, 2025
26 checks passed
emilk added a commit that referenced this pull request Aug 15, 2025
* Closes <#7397>
* [X] I have followed the instructions in the PR template
I do admit I got a peak NixOS `RequestDeviceError` and deemed it
entirely not worth it to think about that.

#5411 broke rotation of multi-line
`TextShape`s because `PlacedRow::pos` was no longer being rotated, so
let's rotate it.

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
Co-authored-by: Lucas Meurer <[email protected]>
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
* Closes <emilk#7397>
* [X] I have followed the instructions in the PR template
I do admit I got a peak NixOS `RequestDeviceError` and deemed it
entirely not worth it to think about that.

emilk#5411 broke rotation of multi-line
`TextShape`s because `PlacedRow::pos` was no longer being rotated, so
let's rotate it.

---------

Co-authored-by: Emil Ernerfeldt <[email protected]>
Co-authored-by: Lucas Meurer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is broken epaint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Multi-line TextShape rotation newly broken in 0.32

3 participants