Results 195 comments of Rob Parrett

@mockersf Do you think we really need to show off non-repeating textures in this example? Could we just strip that part of the example out?

I think that's sort of inherent in an example where there all these extra steps to do a repeating texture. The important part in my eyes is showing users how...

I will try to rebase this soon.

> Maybe we also want to add a DefaultSpatialScale setting? I felt like doing this would be sort of ugly -- maybe requiring spatial_scale to be an Option, so I...

Yeah, I guess things would work more or less like before this PR for configuring the default scale, but `PlaybackSettings` would need to use an `Option` to make > used...

It was easier to just re-implement the changes than it was to fix merge conflicts, so I did that and added `DefaultSpatialScale`. That wasn't a super small change, so I'd...

Thanks for the reminder. The PR description should be up-to-date now. While updating the migration guide I found an opportunity to make this slightly less breaking by taking a `SpatialScale`...

> Should a link always point to the doc? [Most links are pointed to the source code here](https://docs.rs/bevy/latest/bevy/transform/components/struct.Transform.html) This seems like a bug / collision in the `Transform` docs. Only...

I believe this would be fixed by https://github.com/bevyengine/bevy/pull/7948. Would you mind checking?

I can confirm that #7948 fixes the example above.