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.

Sign up for GitHub

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

Calculate the text alignment offset #6724

Closed
wants to merge 22 commits into from
Closed

Calculate the text alignment offset #6724

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
e6d0947
fixed extract_text_uinodes offset alignment bug
ickshonpe Nov 22, 2022
6775b48
Merge branch 'bevyengine:main' into text-wrap-justify
ickshonpe Nov 22, 2022
bcb16c0
Update crates/bevy_ui/src/render/mod.rs
ickshonpe Nov 22, 2022
a1ec7ac
Update mod.rs
ickshonpe Nov 22, 2022
6b9473a
cargo fmt --all
ickshonpe Nov 22, 2022
feebad6
fix alignment offset being multiplied by scale twice
ickshonpe Nov 22, 2022
28efa7c
cargo fmt --all
ickshonpe Nov 22, 2022
b7d9aa3
removed the comment
ickshonpe Nov 24, 2022
75a2d8c
added text alignment example
ickshonpe Nov 24, 2022
9630c33
add example metadata
ickshonpe Nov 24, 2022
3e1b211
tiny cosmetic improvement
ickshonpe Nov 24, 2022
8d7dfa3
cargo fmt -all
ickshonpe Nov 24, 2022
8334276
fixes and formatting for example
ickshonpe Nov 24, 2022
ec8d739
removed example
ickshonpe Nov 24, 2022
830a5dc
simplified the example
ickshonpe Nov 24, 2022
d6e736b
remove scaling from example
ickshonpe Nov 24, 2022
d111065
Alignment_offset *does* need to multiplied by the scale factor.
ickshonpe Nov 25, 2022
b9fb69c
Multiply by ui_node.size() by the scale factor before calculating offset
ickshonpe Nov 25, 2022
4bd17ef
Merge branch 'calculate-text-alignment' of https://github.com/ickshon…
ickshonpe Nov 25, 2022
0a86a23
forgot to remove "* scale_factor"
ickshonpe Nov 25, 2022
7bb6e52
simplified example
ickshonpe Nov 26, 2022
e7de76b
cargo fmt
ickshonpe Nov 26, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Alignment_offset *does* need to multiplied by the scale factor.
  • Loading branch information
@ickshonpe
ickshonpe committed Nov 25, 2022
commit d111065ecd88232352e67e01246264686a6d2d83
2 changes: 1 addition & 1 deletion crates/bevy_ui/src/render/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,7 @@ pub fn extract_text_uinodes(

let extracted_transform = global_transform.compute_matrix()
* Mat4::from_scale(Vec3::splat(scale_factor.recip()))
* Mat4::from_translation((alignment_offset + text_glyph.position).extend(0.));
* Mat4::from_translation((alignment_offset * scale_factor + text_glyph.position).extend(0.));

extracted_uinodes.uinodes.push(ExtractedUiNode {
stack_index,
Expand Down